Skip to content
Snippets Groups Projects

feat: simpler urls with permalinks

Merged Rémi PAILHAREY requested to merge feat/simpler-urls into dev

Related to [5][SEO] - simplification des url (#456 - closed)

What does this MR do and why?

  • Use permalinks instead of IDs for users, structures and articles
  • Renamed lots of links
    • login => connexion
    • form => formulaire
    • profile => profil
    • edit-structure => edition-structure
    • structure-members-management => gestion-membres
    • structures-management => gestion-structures

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally (or on alpha)

:warning: :warning: :warning: THIS MR is linked to pamn_server!382 (merged) :warning: :warning: :warning:

Checkout to feat/simpler-urls for client and checkout to feat/user-and-structure-permalinks for server

Run migrations to generate permalinks for all users and structures

Navigate through the whole app !


MR acceptance checklist

To be completed by the chosen reviewer.

Quality Bugs - Code Smells

  • For the code that this change impacts, I believe that the automated tests validate functionality that is highly important to users. If the existing automated tests do not cover this functionality, I have added the necessary additional tests or I have added an issue to describe the automation testing gap and linked it to this MR.
  • I have made sure that the sonar quality coverage is up to standards.
  • I have considered the impact of this change on the front-end, back-end, and database portions of the system where appropriate and applied.
  • I have tested this MR in all supported browsers or determined that this testing is not needed.
  • I have confirmed that this change is backwards compatible across updates (migrate up needs a migrate down), or I have decided that this does not apply.

Performance, reliability and availability

  • I am confident that this MR does not harm performance, or I have asked a reviewer to help assess the performance impact.
  • I have considered the scalability risk based on future predicted growth.

Documentation

  • The MR is named after the desired squash commit to feed the changelog linked to the current milestone.
  • I have added/updated documentation (also updated if the changes feature a deprecation) or I have decided that documentation changes are not needed for this MR.

Security Security Rating

  • I have confirmed that if this MR does not contains any sensitive informations hidden in the changes.

Related to #456 (closed)

Edited by Rémi PAILHAREY

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading