Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feat/288 mock api #298

Merged
merged 44 commits into from
Aug 13, 2024
Merged

Feat/288 mock api #298

merged 44 commits into from
Aug 13, 2024

Conversation

ndharmateja
Copy link
Collaborator

@ndharmateja ndharmateja commented Aug 11, 2024 •
edited by Monstarrrr
Loading

  1. modularize express app
  2. setup skeleton routes for the current ones
  3. setup authentication and error handling
  4. setup prettier config

Monstarrrr and others added 26 commits July 30, 2024 03:13
@Monstarrrr
Added type to createPost()
3e4ed6e
@Monstarrrr
Merge branch 'main' into feat/5_rebuttals-submission to remain up-to-…
b92b39e 
…date with 'main'.
@Monstarrrr
Saving progress to switch branch
0134097
@Monstarrrr
[WIP] Mock API connection to DB and endpoints
8b02dc7
@Monstarrrr
Fixed packages
ed0618d
@Monstarrrr
Uncommented code that was temporarily commented for testing
9781e6e
@ndharmateja
install ts-node-dev and add dev script
b854ccf
@ndharmateja
add prettier rc
5d57038
@ndharmateja
install express-async-errors
69620b0
@ndharmateja
add routers
76b75ab
@ndharmateja
add middleware
e8ea16c
@ndharmateja
refactor registerUser controller into a separate fn
a333723
@ndharmateja
create express app
57a75e3
@ndharmateja
clean index.ts
031a73c
@ndharmateja
add custom error
09a8381
@ndharmateja
add config
4c7929d
@ndharmateja
refactor imports
6b73357
@ndharmateja
install @types/node
0a7513d
@ndharmateja
rename authenticator middleware
06a9000
@ndharmateja
add logger fns
66778db
@ndharmateja
refactor route exports
57aa1bc
@ndharmateja
add not found and logger mws
3f49363
@ndharmateja
add mws to the app
7723cfa
@ndharmateja
fix bug in authentication mw order
918086e
@ndharmateja
remove log and update error message
0fc7e8a
@ndharmateja
Merge branch 'main' into feat/288_mock-api
bad0c0e
@ndharmateja ndharmateja self-assigned this Aug 11, 2024
@ndharmateja ndharmateja linked an issue Aug 11, 2024 that may be closed by this pull request
Initialize Mock API #288
Closed
@vercel Vercel
Copy link

vercel bot commented Aug 11, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rebutify ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 13, 2024 5:59pm

@ndharmateja ndharmateja marked this pull request as ready for review August 11, 2024 21:05
@ndharmateja ndharmateja added the [ mock-backend / Anything related to the Mock API label Aug 11, 2024
Monstarrrr
Monstarrrr requested changes Aug 12, 2024
View reviewed changes
backend-mock/.prettierrc Outdated Show resolved Hide resolved
backend-mock/package.json Show resolved Hide resolved
backend-mock/src/utils/middleware.ts Outdated Show resolved Hide resolved
backend-mock/src/utils/middleware.ts Show resolved Hide resolved
backend-mock/src/controllers/auth-users.ts Outdated Show resolved Hide resolved
backend-mock/src/utils/errors.ts Outdated Show resolved Hide resolved
backend-mock/src/utils/config.ts Outdated Show resolved Hide resolved
backend-mock/src/utils/logger.ts Outdated Show resolved Hide resolved
backend-mock/package.json Outdated Show resolved Hide resolved
backend-mock/package.json Outdated Show resolved Hide resolved
@Monstarrrr Monstarrrr mentioned this pull request Aug 13, 2024
Closed
@Monstarrrr
Copy link
Owner

Just tested with client, it works 🚀

@Monstarrrr Monstarrrr merged commit 43009da into main Aug 13, 2024
5 checks passed
@Monstarrrr Monstarrrr deleted the feat/288_mock-api branch August 13, 2024 18:02
@ndharmateja
Copy link
Collaborator Author

Just tested with client, it works 🚀

amazing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Monstarrrr Monstarrrr Monstarrrr approved these changes

Assignees

@ndharmateja ndharmateja

Labels
[ mock-backend / Anything related to the Mock API
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix typescript error in mock-api branch Handle permissions for mock API routes Initialize Mock API
2 participants
@ndharmateja @Monstarrrr