stable

Clone or download

Read-only

Migrate merge and error modals

Part of story #10410 - Have burning parrot pull requests How to test: --> Error modal should be properly displayed when there is an error (#protip: to get one, load the closed PRs from the homepage, then click on a pull request, go back using the browser's button -> "Oh snap!"). --> Merge modal is displayed properly --> The merge is done when you click on "continue merging" --> The PR is not merged if you cancel Change-Id: I5f2a72058d1445a7e8c634976cef4c86a46fa315

Modified Files

Name
M plugins/pullrequest/www/scripts/po/fr.po +6 −1 Go to diff View file
M plugins/pullrequest/www/scripts/po/template.pot +5 −0 Go to diff View file
M plugins/pullrequest/www/scripts/src/app/app.js +3 −4 Go to diff View file
M plugins/pullrequest/www/scripts/src/app/error-modal/error-modal-controller.js +3 −8 Go to diff View file
M plugins/pullrequest/www/scripts/src/app/error-modal/error-modal-service.js +23 −17 Go to diff View file
M plugins/pullrequest/www/scripts/src/app/error-modal/error-modal.tpl.html +34 −14 Go to diff View file
M plugins/pullrequest/www/scripts/src/app/overview/_overview.scss +0 −1 Go to diff View file
D plugins/pullrequest/www/scripts/src/app/overview/merge-modal/_merge-modal.scss +0 −5 Go to diff View file
M plugins/pullrequest/www/scripts/src/app/overview/merge-modal/merge-modal-controller.js +8 −6 Go to diff View file
M plugins/pullrequest/www/scripts/src/app/overview/merge-modal/merge-modal-service.js +19 −7 Go to diff View file
M plugins/pullrequest/www/scripts/src/app/overview/merge-modal/merge-modal.tpl.html +21 −9 Go to diff View file
M plugins/pullrequest/www/scripts/src/app/overview/overview-controller.js +12 −5 Go to diff View file
M plugins/pullrequest/www/scripts/webpack.config.js +4 −2 Go to diff View file
M tools/utils/scripts/webpack-aliases.js +1 −0 Go to diff View file