-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
UGC-4233 - Fixed multitude of problems with UsingData #7
Conversation
Would be good to get the CI working (asking someone with admin perms to enable it?), it seems the current parser tests pass on this branch:
Maybe we can add some more test cases that would have failed before? |
Apparently I'm the admin, so I'll see what I can do |
84a2983
to
a81a820
Compare
35d3889
to
cb31d2a
Compare
e1d0040
to
2310e30
Compare
2310e30
to
953807f
Compare
During upgrade to 1.39 we did a major refactor of the extension, and it broke in multiple ways:
After spending some time trying to pinpoint the root causes and fix those problems I've decided to instead revert the codebase to it's pre-upgrade state and fix issues starting there, which proved significantly easier.
the diff that handles this part is folowing: a3f162e