Skip to content

Fixes here and there#75

Merged
StrangeGirlMurph merged 6 commits intomainfrom
maintenance
Mar 2, 2026
Merged

Fixes here and there#75
StrangeGirlMurph merged 6 commits intomainfrom
maintenance

Conversation

@StrangeGirlMurph
Copy link
Copy Markdown
Member

  • delete proposals after the jam
  • set the eventID for scheduled events to update

@StrangeGirlMurph
Copy link
Copy Markdown
Member Author

Before we merge I should probably also add an announcement when a jam gets extended. For now I shall sleep though

@StrangeGirlMurph
Copy link
Copy Markdown
Member Author

We should get away without having to do any data migration from deleting proposal.used right? I mean the values can just rot there

@Chicken
Copy link
Copy Markdown
Contributor

Chicken commented Mar 1, 2026

Need a migration to delete the proposals that are already marked used.

@StrangeGirlMurph
Copy link
Copy Markdown
Member Author

I figured I can just manually delete the proposals that we already had

@StrangeGirlMurph
Copy link
Copy Markdown
Member Author

Okay this should be it for this PR. I am okay with testing in prod. Let me know if you find a bug or something

Copy link
Copy Markdown
Contributor

@Chicken Chicken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

myeah seems fine, just remove the package-lock.json, i may have done an executive decision in 'do this and that' to move to pnpm

@StrangeGirlMurph
Copy link
Copy Markdown
Member Author

fine. I'll add it to the gitignore

@StrangeGirlMurph StrangeGirlMurph merged commit ab1f744 into main Mar 2, 2026
4 checks passed
@StrangeGirlMurph StrangeGirlMurph deleted the maintenance branch March 2, 2026 12:09
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