Skip to content

feat(routes): Help Requests API Endpoints with validation#35

Merged
prdai merged 7 commits intob3-competition:mainfrom
Methika1234:feat-27-help-requests-api-endpoints
Jul 5, 2025
Merged

feat(routes): Help Requests API Endpoints with validation#35
prdai merged 7 commits intob3-competition:mainfrom
Methika1234:feat-27-help-requests-api-endpoints

Conversation

@Methika1234
Copy link
Copy Markdown
Contributor

close: #27

@Methika1234 Methika1234 requested a review from prdai June 30, 2025 11:35
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

try to match this what is in @Thilina2468 's PR #34

Comment thread src/routes/requests.js Outdated
Comment thread src/routes/requests.js

router.get('/', async (_req, res) => {
try {
const snapshot = await requestsCol.get();
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

the storage needs to be created and then their reference ids or something should be in the atttachements in the firestore btw like a list of strings which has ids which we can refer to in the storage...

Comment thread src/routes/requests.js
}
});

router.put('/:requestId', validate(updateRequestSchema), async (req, res) => {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

here also you have to integrate the firebase storage as required as well...

- Removed the old requests.js route file.
- Introduced helpRequest.js route file with updated request handling logic.
- Updated app.js to use the new helpRequest router.
- Enhanced attachment handling in the new route.
@prdai prdai merged commit e2023bd into b3-competition:main Jul 5, 2025
2 of 4 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.

Help Requests API Endpoints

2 participants