Skip to content

Store View: Extract team status into separate block **(by Jo)**

Chris Oelmueller requested to merge chris/salvage-989 into master

Squashed the original MR !989 (closed) to avoid a huge merge commit

Closes !989 (closed)

What does this MR do?

As the teamlist is collapsed in stores after !983 (merged) got merged, the menu of switching the store join status isn't as present anymore on mobile.

Since this is an option that some users do need to locate, it should be placed on its own, not as appendix to the barely-related list of team members.

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

Works fine locally (the loading indicator keeps spinning, but that also happened before these changes)

Links to related issues

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 bot
  3. Open store page
  4. Change "team status" setting
  5. Reload page, view store on map, in different account/browser etc to confirm the values are changed

Screenshots (if applicable)

This is a frontend change.

status_menu

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

Merge request reports