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.

Sign up for GitHub

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

Jump to bottom

Fix the export link of ASM movements #8126

Merged
merged 1 commit into from Jul 21, 2017
Merged

Fix the export link of ASM movements #8126

merged 1 commit into from Jul 21, 2017

Conversation

hibatallahAouadni
Copy link
Contributor

Questions Answers
Branch? 1.6.1.x
Description? in ASM movements, the export button doesnt work because of "&amp" in URL.
Type? bug fix
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? http://forge.prestashop.com/browse/PSCSX-9180
How to test? BO -> ASM (Advanced Stock Management) -> Stock Movement, choose one warehouse in the filter select, then try to click on the export button, the CSV file will be generated.

@basma-yangui-prestashop basma-yangui-prestashop added the Waiting for QA Status: action required, waiting for test feedback label Jul 11, 2017
dave-lw
dave-lw approved these changes Jul 11, 2017
View reviewed changes
Copy link
Contributor

@dave-lw dave-lw left a comment

Choose a reason for hiding this comment

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

working with fix, thank you.

@maximebiloe maximebiloe added this to the 1.6.1.16 milestone Jul 11, 2017
@maximebiloe
Copy link
Contributor

Approved by @dave-ps

@prestonBot prestonBot added the QA ✔️ Status: check done, code approved label Jul 21, 2017
@maximebiloe maximebiloe removed the Waiting for QA Status: action required, waiting for test feedback label Jul 21, 2017
@maximebiloe maximebiloe merged commit 781111b into PrestaShop:1.6.1.x Jul 21, 2017
@maximebiloe
Copy link
Contributor

Thanks @hibatallahAouadni

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dave-lw dave-lw dave-lw approved these changes

Assignees
No one assigned
Labels
QA ✔️ Status: check done, code approved
Projects
None yet
Milestone
1.6.1.16
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@hibatallahAouadni @maximebiloe @dave-lw @prestonBot @basma-yangui-prestashop