Skip to content
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

[v3.2] Merge pull request #4707 from cpfergus1/cpfergus1/fix-broken-image-upload-js #4708

Merged
merged 1 commit into from
Nov 9, 2022

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Nov 9, 2022

Backport

This will backport the following commits from master to v3.2:

Questions ?

Please refer to the Backport tool documentation

The was changed by mistake to `$.ajaxSetup.xhr()` and broke the image
upload drop field. The error was missed because there were not any
existing tests for the image drop field so the script was not being
properly tested.

Although `$.ajaxSetup().xhr()` could work in this situation, it will
be fully reverted to original due to inability to find source of
deprecation cited in commit that changed this function.

(cherry picked from commit 58fcaae)
Copy link
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @cpfergus1 and backporting bot!

@kennyadsl kennyadsl added type:bug Error, flaw or fault changelog:solidus_backend Changes to the solidus_backend gem labels Nov 9, 2022
@waiting-for-dev waiting-for-dev merged commit a56bb5c into v3.2 Nov 9, 2022
@waiting-for-dev waiting-for-dev deleted the backport/v3.2/pr-4707 branch November 9, 2022 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:solidus_backend Changes to the solidus_backend gem type:bug Error, flaw or fault
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants