Devin Cox
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.