Skip to content

Comments

Event Tags & DB refactor to camel case#169

Merged
ashleynhs merged 6 commits intomainfrom
Kai/event-tags
Feb 24, 2026
Merged

Event Tags & DB refactor to camel case#169
ashleynhs merged 6 commits intomainfrom
Kai/event-tags

Conversation

@lauren1703
Copy link
Contributor

Overview

  • Merged Kai's event tag model/endpoints
  • Added new migration to rename all table/column names to camel case (both resell-test db and resell-dev db should match now)
  • Debugged codebase/tests to account for camel case refactor. I was running into deadlock issues so I changed maxWorkers to 1 in jest.config.ts
  • Fixed super admin email tests to add fake super admin email to env (not sure if this functionality works in prod because it checks for admin emails in .env but apparently aayush deleted it? idk)
  • Added event tag tests to PostTest suite

Test Coverage

  • All tests pass

ahappyphrog and others added 5 commits February 22, 2026 18:14
Add remove tags, new schema, tests/factories, swagger
- Dropped category column from Post model
- Changed join tables to camel casing in models
CLAUDE.md Outdated
Copy link

Choose a reason for hiding this comment

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

I don't think we want this in the repo?

Copy link
Contributor

Choose a reason for hiding this comment

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

Agreed

@lauren1703 lauren1703 requested a review from JoshD94 February 23, 2026 21:55
Copy link

@JoshD94 JoshD94 left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@ashleynhs ashleynhs left a comment

Choose a reason for hiding this comment

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

looks good!

@ashleynhs ashleynhs merged commit ec94e29 into main Feb 24, 2026
3 checks passed
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.

4 participants