Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade mapper frontend to Svelte 5 Runes #1846

Merged
merged 15 commits into from
Oct 31, 2024
Merged

Conversation

spwoodcock
Copy link
Member

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation
  • πŸ§‘β€πŸ’» Refactor
  • βœ… Test
  • πŸ€– Build or CI
  • ❓ Other (please specify)

Describe this PR

  • I made the code a bit more modular.
  • Upgraded to Svelte 5, so instead of writable svelte/store, we use global reactivity using Runes.

More details:

Checklist before requesting a review

[optional] What gif best describes this PR or how it makes you feel?

@spwoodcock spwoodcock added enhancement New feature or request priority:low Backlog of tasks that will be addressed in time frontend Related to frontend code effort:low Likely a few hours labels Oct 31, 2024
@spwoodcock spwoodcock self-assigned this Oct 31, 2024
@github-actions github-actions bot added the devops Related to deployment or configuration label Oct 31, 2024
@spwoodcock spwoodcock merged commit c597522 into development Oct 31, 2024
2 checks passed
@spwoodcock spwoodcock deleted the feat/upgrade-svelte-5 branch October 31, 2024 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devops Related to deployment or configuration effort:low Likely a few hours enhancement New feature or request frontend Related to frontend code priority:low Backlog of tasks that will be addressed in time
Projects
Development

Successfully merging this pull request may close these issues.

1 participant