Skip to content

Fix issue with user role on dashboard

Michael Aydinbas requested to merge 52-pmayd-fix-user-role-on-dashboard into master

Closes #52 (closed)

What does this MR do?

The user role on the foodsaver dashboard was wrongly associated with the home district. This resulted in texts like 'Botschafter für xyz". Although user is an ambassador for some region x, this has not to be his/her home district.

Issue was solved by deleting this sentence completely as was argued that the user should be aware of their roles anyway. Instead, I added more information about the user's activity. Now we have three informations: Number of fetch counts, total fetch weight and the home district.

_52_dashboard_fix

If there is no fetch weight or fetch count, only the home district is shown.

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

Very, only text/shown information was changed, no internal logic

Links to related issues

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))
Edited by Peter Tönnies

Merge request reports