Skip to content

Conversation

@opyh
Copy link
Member

@opyh opyh commented Sep 8, 2025

πŸ–ŠοΈ Description

πŸ’‘ What? Why? How?

This fixes loading the mapping event list.

πŸ“Ž Related Ticket

https://app.asana.com/1/1200321573365931/project/1208833037492986/task/1211034077235092

⚠️ Creation checklist

Before creating this merge request, go over the following checklist (click to expand).
Remove any items that are not applicable.

πŸ’ͺ I have tested my code
  • A new E2e playwright test covers this feature / A new test that reproduces the bug passes now.
  • The feature deployment works.
  • The automated tests are passing.
  • I have manually tested this feature
    • on mobile
    • by using keyboard-only navigation
    • with a screen reader (VoiceOver is fine)
    • in Chrome
    • in Firefox
    • in Safari
🧼 I have cleaned up my code - [x] I have removed dependencies that were just for testing. - [x] I have removed debug logging. - [x] My code does not generate new warnings. - [x] My code does not depend on new vulnerable packages. - [ ] The commit messages are precise and make sense (rebase the PR with `--interactive` if applicable, keeping commits in sensible chunks if possible).
πŸ” I have performed a self-review of my code
  • My code is self-documenting or has links to necessary documentation.
  • New function and variables names can be understood by new developers with basic project knowledge.
  • The feature fits the design.
✨ I have created a nice pull request
  • It has a clear title.
  • It follows the template, has a clear description and testing instructions if needed.
  • It references applicable Asana tickets.
  • It targets the right branch.
  • I removed not applicable sections of the PR template.
  • [optional] I added a GIF of my favorite animal to the PR description to lighten the mood of my colleagues.
πŸ“ I updated the documentation
  • I updated the documentation in this repository.
  • I updated the tech manual.
  • I updated the manual testing plan of the app so that it includes a testing flow for this feature.

πŸ” Reviewing

When reviewing this merge request, here are some things to keep in mind:

πŸ”¬ Testing instructions

@opyh opyh marked this pull request as ready for review September 8, 2025 08:30
@opyh opyh requested a review from a team as a code owner September 8, 2025 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants