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

Web: Avoid top-level Event #4142

Merged
merged 2 commits into from
Feb 24, 2025
Merged

Conversation

madsmtm
Copy link
Member

@madsmtm madsmtm commented Feb 24, 2025

This allows removing the top-level Event enum in the future, and makes it clearer which events the backend actually emits.

@madsmtm madsmtm added DS - web S - maintenance Repaying technical debt labels Feb 24, 2025
@madsmtm madsmtm requested a review from daxpedda as a code owner February 24, 2025 06:29
@madsmtm madsmtm force-pushed the madsmtm/web-avoid-top-level-event branch from 063168f to dc56301 Compare February 24, 2025 06:43
@madsmtm madsmtm force-pushed the madsmtm/web-avoid-top-level-event branch from dc56301 to 9656796 Compare February 24, 2025 07:00
@madsmtm madsmtm changed the title web: Avoid top-level Event Web: Avoid top-level Event Feb 24, 2025
@kchibisov kchibisov merged commit 4d6fe7e into master Feb 24, 2025
57 checks passed
@kchibisov kchibisov deleted the madsmtm/web-avoid-top-level-event branch February 24, 2025 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DS - web S - maintenance Repaying technical debt
Development

Successfully merging this pull request may close these issues.

2 participants