Skip to content

Comments

Feat/integrate data schema#99

Merged
sristy17 merged 2 commits intosristy17:dwoc-2025from
AritraDey-Dev:feat/integrate-data-schema
Feb 7, 2025
Merged

Feat/integrate data schema#99
sristy17 merged 2 commits intosristy17:dwoc-2025from
AritraDey-Dev:feat/integrate-data-schema

Conversation

@AritraDey-Dev
Copy link
Contributor

@AritraDey-Dev AritraDey-Dev commented Feb 5, 2025

Fixes Issue #97

Changes proposed

  1. User Model Update:

    • Implement a pre-save hook in the user model to automatically calculate the age based on the dob (date of birth) provided during user signup.
    • Remove the required constraint from the age field in the schema to allow it to be calculated dynamically.
  2. Database Integration:

    • Ensure that the calculated age is saved in the database when a new user is created.
  3. UI Adjustments:

    • Confirm that the signup form collects the dob input from users, which is necessary for age calculation.
  4. Testing:

    • Test the signup process to ensure that the age is correctly calculated and stored in the database without any errors.

Check List (Check all the applicable boxes)

  • My code follows the code style of this project.
  • My change requires changes to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Screencast.from.2025-02-05.13-26-13.webm

…nted a pre-save hook to calculate age from dob and save it in the database.
…nted a pre-save hook to calculate age from dob and save it in the database.
Copy link
Owner

@sristy17 sristy17 left a comment

Choose a reason for hiding this comment

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

Great job, @AritraDey-Dev! 🎉 Thank you for submitting your pull request. Your contribution is valuable and we appreciate your efforts to improve our project.

We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

@AritraDey-Dev
Copy link
Contributor Author

@pooranjoyb @sristy17 here is the PR for the issue #99.Please verify and lf me know if you feel any changes required.

@pooranjoyb
Copy link
Collaborator

@pooranjoyb @sristy17 here is the PR for the issue #99.Please verify and lf me know if you feel any changes required.

Cool LGTM! @sristy17 Please review and merge if fine.

@pooranjoyb pooranjoyb requested a review from sristy17 February 5, 2025 17:59
@sristy17 sristy17 added dwoc-2025 Issue under DWOC 2025 level 4 labels Feb 7, 2025
@sristy17 sristy17 merged commit bf4d96b into sristy17:dwoc-2025 Feb 7, 2025
1 check passed
@AritraDey-Dev AritraDey-Dev deleted the feat/integrate-data-schema branch April 17, 2025 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dwoc-2025 Issue under DWOC 2025 level 4

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants