Devin Cox

Results 11 comments of Devin Cox

Hello everyone! I wanted to check in and see if there was any work being done on this issue. I am new to this repo but I can try taking...

Resolved the issue commented on above. Interestingly, the failure was actually caused when using a model with casing like this: `FooBar` The uppercase results in `data-model-ref checking` `foobar` against `foo...

Hey @nessita, thank you for taking a deep dive into this! I was not aware of the m2m issue, but that makes sense. I can set aside some time next...

Just commenting on this, I've been pretty busy lately so haven't had much time to fix this. I did have a chance to peek at the failed test, and it...

Altered the approach to add the context variable for `RelatedWidgetWrapper` instead of the `Select` widget based on whether or not `allow_multiple_selected` was chosen. This still works for the bug in...

> I'm a little lost on how the latest proposal matches the above, could you elaborate a bit? Thank you! Wording was a bit awkward, the previous approach added `data-context`...

Hey @nessita , altered the approach to only add `data-context` if the widget is not an instance of `AutocompleteMixin`. Similar concept but a cleaner approach to not add it when...

> Hey @devin13cox, thank you for your continued and steady contributions! > > I haven't tried this yet but I think this is not the proper solution, in my [original...

Thank you for the reviews @sarahboyce @nessita and sorry about the delay! Was on vacation and missed the update on this. Should be good to go.

Is it possible to re-run the checks? I'm fairly certain this change does not impact cache tests.