Last modified: 2014-09-01 12:58:46 UTC
In order to wrap up frontend changes sonner rather than later, some corners were cut. The plan was to avoid finding specific nodes in JS code, but rather have selectors etc in the templates. Try searching for .closest( 'form' ) in flow-board.js, for example ;) Here's an incomplete list of stuff that definitely needs fixing. Please help complete this list & fix the code ;) * Error handling: FlowBoardComponent.UI.events.interactiveHandlers.apiRequest has generic error handling, but always attached the error message to $this.closest( 'form' ) - even though errors can occur without relevant forms (e.g. FlowBoardComponent.UI.events.apiHandlers.board, which has to manually attach the errors because the generic error handling assumes form...)
In backlog: https://trello.com/c/vsKwxixR
Change 146785 had a related patch set uploaded by Matthias Mullie: Fix error handling https://gerrit.wikimedia.org/r/146785
Change 146785 merged by jenkins-bot: Fix error handling https://gerrit.wikimedia.org/r/146785
All patches mentioned in this report were merged or abandoned - is there more work left to do here (if yes: please reset the bug report status to NEW or ASSIGNED), or can you close this ticket as RESOLVED FIXED?
