Skip to content

Fix invitation max team size on approval and enforce more eslint rules#49

Merged
olix3001 merged 2 commits intomainfrom
fix-teams-and-add-rules
Feb 24, 2026
Merged

Fix invitation max team size on approval and enforce more eslint rules#49
olix3001 merged 2 commits intomainfrom
fix-teams-and-add-rules

Conversation

@olix3001
Copy link
Contributor

This PR fixes check for team size and add automated tests for it:
Now invite fails when action is taken whilst team size is >= max event allowed size

This PR also introduces more styling rules and most importantly - Copyright notice check under the following directories:

  • app/
  • config/
  • database/
  • start/
  • tests/

Due to those changes there is a lot of reformatting included.

@olix3001 olix3001 requested a review from InfoX1337 February 24, 2026 22:33
Copy link
Contributor

@InfoX1337 InfoX1337 left a comment

Choose a reason for hiding this comment

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

tl-dr.

@InfoX1337 InfoX1337 self-requested a review February 24, 2026 22:37
Copy link
Contributor

@InfoX1337 InfoX1337 left a comment

Choose a reason for hiding this comment

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

We really need to set a max files changed limit on PRs... This is not what I wanted to read at 12AM, although it LGTM.

Copy link
Contributor

Choose a reason for hiding this comment

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

👍🏼 finally won't forget to put the copyright in

@olix3001 olix3001 force-pushed the fix-teams-and-add-rules branch from 1a47d06 to 4a4c3d1 Compare February 24, 2026 22:41
@olix3001 olix3001 merged commit 4a4c3d1 into main Feb 24, 2026
4 checks passed
@olix3001 olix3001 deleted the fix-teams-and-add-rules branch February 24, 2026 22:42
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