Skip to content

Conversation

LightCreator1007
Copy link
Contributor

@LightCreator1007 LightCreator1007 commented Aug 3, 2025

Overview

  1. This PR fixes or fixes part of [Feature Request]: Create Routes to fetch and update user data. #41 .
  2. This PR does the following: adds user routes and fixes fetcing logic in the frontend

Essential Checklist

  • The PR title starts with "Fix #bugnum: ", followed by a short, clear summary of the changes. (If this PR fixes part of an issue, prefix the title with "Fix part of #bugnum: ...".)
  • "Allow edits from maintainers" is checked. (See here for instructions on how to enable it.)
  • The PR is made from a branch that's not called "main/master".

- added user routes to fetch all, fetch single, update, add and delete
  user.

- added a centralized user fetching in frontend.

Closes bsoc-bitbyte#41
Copy link

netlify bot commented Aug 3, 2025

Deploy Preview for ydo-bsoc ready!

Name Link
🔨 Latest commit 39aafce
🔍 Latest deploy log https://app.netlify.com/projects/ydo-bsoc/deploys/68962be651e73100080f23d7
😎 Deploy Preview https://deploy-preview-43--ydo-bsoc.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@LightCreator1007
Copy link
Contributor Author

I tried a few things like using uuid from the auth.users as a pk for user_id in the public.users, and a flask decorator to authnticate user and get the uid. I am not completely sure if i am correct in my implementations and assumptions.
kindly guide if wrong at any point.

@Om-Thorat
Copy link
Contributor

looks great, can you also create a route to get a specific user give a roll number?

@Om-Thorat Om-Thorat added Difficulty : Hard hard level issue BSoC'25 Created for BSoC'25 Backend labels Aug 7, 2025
@LightCreator1007
Copy link
Contributor Author

umm i think i misunderstood something prolly, so we needed a path to get roll number of a user and store it as ID? but i did that in the add user route. This is just an assumption, but can we not explicitly ask the user for roll number and take it in as ID anyways coz the add user route already handles it?

@Om-Thorat
Copy link
Contributor

The roll number can extracted using the mail id, since users will be allowed to login only the institute id.

@Om-Thorat Om-Thorat merged commit eb29b6e into bsoc-bitbyte:main Aug 14, 2025
6 checks passed
@Om-Thorat
Copy link
Contributor

Good Job! @LightCreator1007

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend BSoC'25 Created for BSoC'25 Difficulty : Hard hard level issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants