Change birthdate to age

[allennatang]

Tickets:

#936

Related to dashboard issue: hackmcgill/dashboard#1003

• HAC-

List of changes:

Changed hacker account's birthdate field to age field to match MLH guidelines.

Created function ageValidator() in hackerAPI\middlewares\validators\validator.helper.js to validate that age is between 0 and 100
Commented out getAge function which calculates hackers' ages from their birthdate.

Type of change

• Bug fix (non-breaking change which fixes an issue)

How has this been tested?

The UI side of the dashboard seems to work since you can only input ages from 1 to 99.

Questions for code reviewers?

How to do more testing?

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings
• Listed change(s) in the Changelog
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• I have made corresponding changes to the documentation
• Any dependent changes have been merged and published in downstream modules

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
hacker-api	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Nov 14, 2024 11:54pm

[allennatang]

The most recent commit also contains the changes for this related dashboard issue hackmcgill/dashboard#1013 where I added a country of residence field to the hacker application. In the hackerAPI, I added validation (checking that the field is not empty) and I also added it to the example api data (not sure if that was necessary, but doesn't do anything).

[tektaxi]

unsure why that test is failing, works according to my tests. need to merge for mlh requirements.