Skip to content

Replace hard coded links in MenuAdmin.vue with $url()

mr-kenhoff requested to merge 884-replace-hardcoded-links into master

Closes #884 (closed)

What does this MR do?

Replaced hard coded links with references using $url(). Additionally I sorted the entries in client/src/urls.js alphabetically.

How confident are you it won't break things if deployed?

Very confident. Only touched very few lines and tested them...

How to test

Steps a reviewer can take to verify that this MR does what it says it does e.g.

  1. Checkout branch locally
  2. Login as userorga@example.com
  3. Open the administration menu using the gear symbol on the toolbar.
  4. Check if all the links are working correctly.

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings
Edited by Chris Oelmueller

Merge request reports