search

VandyHacks / vaken

Next-gen hackathon registration system
MIT License
49 stars 12 forks source link

Add asterisks required qs #825

Closed wua3 closed 3 years ago

wua3 commented 3 years ago

Summary

PR does 2 things:

  1. In the hacker application, indicates which questions are required w/ an asterisk. (Already in data entity, so just adding conditional logic in UI.) asterisks

    • Closes #824
    • To replicate, simply run the local instance (npm run dev) and visit http://localhost:8081/application. All sections of the application (Demographic Information, Hacker Application, and the last one, Shipping, which you will see has no required questions) should have asterisks indicating when appropriate the required nature of a question.

  2. Minor clarifications in documentation in steps for setting up local dev environment

codeclimate[bot] commented 3 years ago

Code Climate has analyzed commit d266d327 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (0% is the threshold).

This pull request will bring the total coverage in the repository to 63.4% (0.0% change).

View more on Code Climate.