Skip to content

Fixed store team status which accidentally broke in beta

Johannes requested to merge JO-fixed-store-team-status-sentry-1072094327 into master

What does this MR do?

Fixed store team status which accidentally broke in !933 (merged)
Bug was in beta only!
Sentry issue: https://sentry.io/organizations/foodsharing-ev/issues/1072094327/

The line to check was previously using (int)$_GET['s'] (s for status)
https://gitlab.com/foodsharing-dev/foodsharing/blob/master/src/Lib/Xhr/XhrMethods.php#L1137

store_team_status

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

Should work

Links to related issues

  • Closes an issue before it is being filed ;-)

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 Chris Oelmueller

Merge request reports