Skip to content

Fix team gets cleared when user is checked in#130

Merged
sezanzeb merged 9 commits into
mainfrom
fix-check-in-clears-team
May 31, 2026
Merged

Fix team gets cleared when user is checked in#130
sezanzeb merged 9 commits into
mainfrom
fix-check-in-clears-team

Conversation

@sezanzeb
Copy link
Copy Markdown
Collaborator

@sezanzeb sezanzeb commented May 28, 2026

  • findUsersByIDs loads team and teamRequest now (fixes the problem in the title)
  • admitting and such do partial updates now, to avoid accidentally overwriting team and teamRequest (fixes the problem in the title too)
  • add tests
  • Removed yarn because it was removed from node:alpine it seems, so we would have been required to create another dockerfile for local docker development. Since I am incredibly tired of breaking changes, deprecations, etc. in javascript toolings and wasting my time changing config files to fix javascript projects, vanilla npm it is. Which means I'm changing configurations once again.
  • Removed the small and broken frontend test, so that I can get rid of the enzyme dependencies that were incompatible with our react version
  • Had to upgrade chart.js and fix breaking changes after switching to npm, because npm tells me that there are issues in the dependency tree

@sezanzeb sezanzeb merged commit f9b31e1 into main May 31, 2026
4 checks passed
@sezanzeb sezanzeb deleted the fix-check-in-clears-team branch May 31, 2026 20:16
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.

1 participant