stable
Clone or download
Posting a new thread in forumml is now done via a modal. The legacy Flaming Parrot page is removed. Same behavior is kept (adding/removing CC/attachments). Note: The legacy url is still used to post the message. Since it is not accepting GET anymore, the user that might have bookmarked the page may encounter an [apparently internal server error][0] instead of a "404 Not found". Therefore the legacy url is kept to redirect to the threads list where the user can find the button to post a new thread. Once the POST will use a new url, then we will be able to remove the redirection. Part of request #18846: Switch ForumML to BurningParrot [0]: in that case, the status code is 405 (method not allowed), which is more of a "client error" (4xx) than a "server error" (5xx). However FrontRouter handles it as a server error. Change-Id: I4d179c1d3d5eccc7c535649809f7fcdeef87f326
Modified Files
| Name | ||||
|---|---|---|---|---|
| A | plugins/forumml/build-manifest.json | +9 | −0 | Go to diff View file |
| M | plugins/forumml/include/ForumML/Threads/ThreadsController.php | +1 | −0 | Go to diff View file |
| M | plugins/forumml/include/ForumML/Threads/ThreadsPresenterBuilder.php | +10 | −7 | Go to diff View file |
| M | plugins/forumml/include/WriteMailController.php | +1 | −97 | Go to diff View file |
| M | plugins/forumml/include/forummlPlugin.php | +1 | −1 | Go to diff View file |
| M | plugins/forumml/include/forumml_utils.php | +1 | −1 | Go to diff View file |
| A | plugins/forumml/jest.config.js | +23 | −0 | Go to diff View file |
| M | plugins/forumml/package-lock.json | +305 | −2 | Go to diff View file |
| M | plugins/forumml/package.json | +13 | −3 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread.ts | +47 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread/po/fr_FR.po | +28 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread/src/components/App.test.ts | +65 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread/src/components/App.vue | +96 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread/src/components/CcInput.test.ts | +51 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread/src/components/CcInput.vue | +61 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread/src/components/FileInput.test.ts | +52 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread/src/components/FileInput.vue | +89 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread/src/components/__snapshots__/App.test.ts.snap | +37 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread/src/components/__snapshots__/CcInput.test.ts.snap | +27 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread/src/components/__snapshots__/FileInput.test.ts.snap | +40 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/new-thread/src/helpers/local-vue-for-test.ts | +32 | −0 | Go to diff View file |
| A | plugins/forumml/scripts/vue.shims.d.ts | +23 | −0 | Go to diff View file |
| M | plugins/forumml/site-content/fr_FR/LC_MESSAGES/tuleap-forumml.po | +5 | −12 | Go to diff View file |
| A | plugins/forumml/templates/new-thread-modal.mustache | +54 | −0 | Go to diff View file |
| M | plugins/forumml/templates/threads-empty-state.mustache | +10 | −4 | Go to diff View file |
| M | plugins/forumml/templates/threads.mustache | +15 | −6 | Go to diff View file |
| M | plugins/forumml/tests/unit/ForumML/Threads/ThreadsControllerTest.php | +10 | −0 | Go to diff View file |
| A | plugins/forumml/tests/unit/WriteMailControllerTest.php | +48 | −0 | Go to diff View file |
| M | plugins/forumml/themes/BurningParrot/css/_forumml.scss | +1 | −0 | Go to diff View file |
| A | plugins/forumml/themes/BurningParrot/css/_new-thread.scss | +56 | −0 | Go to diff View file |
| A | plugins/forumml/tsconfig.json | +9 | −0 | Go to diff View file |
| M | plugins/forumml/webpack.common.js | +20 | −1 | Go to diff View file |