Skip to content

updated storage handling for map

Closes #1598 (closed)

What does this MR do?

Updated map storage handling, removed prefix ('map'). Added prefix 'map' to the map specific keys (for storage) With these changes the state of the chats is stored without prefix and therefore in the same key as for all other pages.

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

very

Links to related issues

How to test

  1. Checkout branch locally
  2. Login as foodsaver
  3. Open Map
  4. Open a chat, zoom/drag the map
  5. switch to another site (e.g. Dashboard)
  6. Test if the same chat is open, open another chat
  7. Switch back to map
  8. Test if the state related to zoom/location is restored
  9. Test if the last opened chat is still open

Screenshots (if applicable)

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • set a "for:" label to indicate who will be affected by this change
  • added to the next milestone (see https://gitlab.com/foodsharing-dev/foodsharing/-/milestones, unless it has a "for:Dev" label)
  • added an entry to CHANGELOG.md
  • added a short text that can be used in the release notes
  • Once your MR has been merged, you are responsible to create a testing issue in the Beta Testing forum: https://foodsharing.de/?page=bezirk&bid=734&sub=forum. Please change the MRs label to "state:Beta testing".
    • Consider writing a detailed description in German.
    • Describe in a few sentences, what should be tested from a user perspective.
    • Also mention different settings (e.g. different browsers, roles, ...) how this change can be tested.
    • Be aware, that also non technical people should understand.

Release notes text in German

Karte speichert die Info welche Chats geöffnet werden sollten nicht mehr getrennt von allen anderen foodsharing Seiten.

Closes #1598 (closed)

Edited by florianunsinn

Merge request reports