Skip to content

Refactoring and final adjustments for Milestone 2#47

Merged
IsaacIsrael merged 14 commits into
mainfrom
final-adjustments
Apr 26, 2026
Merged

Refactoring and final adjustments for Milestone 2#47
IsaacIsrael merged 14 commits into
mainfrom
final-adjustments

Conversation

@IsaacIsrael
Copy link
Copy Markdown
Owner

  • refactor: rename authentication.getAuthenticatedUser() to authentication.getUser()
  • refactor: explicitly return in /api/v1/status GET handler
  • test: standardize test descriptions with backticks (`)
  • test: replace hardcoded http://localhost:3000 with ${webserver.origin}
  • chore: move dotenv and dotenv-expand to devDependencies
  • refactor: remove async from setSessionCookie() and clearSessionCookie()
  • refactor: inline createRouter() chain in all API route exports
  • test: clear database and run migrations in GET /api/v1/status tests
  • feat: add sameSite=Lax to controller.setSessionCookie()
  • test: olerate small timestamp drift in POST /api/v1/sessions tests
  • chore: bump project dependencies within minor and patch releases
  • chore: upgrade major dependencies that required no code changes
  • chore: upgrade next, react and eslint to latest compatible versions

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clone-tabnews Ready Ready Preview, Comment Apr 26, 2026 8:29am

@IsaacIsrael IsaacIsrael merged commit e36c1ae into main Apr 26, 2026
6 checks passed
@IsaacIsrael IsaacIsrael deleted the final-adjustments branch April 26, 2026 08:40
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