Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[M1_TR-209] Frontend for M1_TR-4 #16

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

jasonchuasun
Copy link
Contributor

@jasonchuasun jasonchuasun commented Sep 12, 2023

Related Issue

WIP

  • TODO: update BE and FE validation - tags and work_schedules are no longer required in form submission
  • TODO: add error message for PersonInCharge if not filled

Context

  • integrate job registration feature
  • add Frontend validation on selected fields

Commands:

  • startup docker
    docker-compose up -d

  • double check .env file if the DATABASE_URL is correct
    DATABASE_URL="postgresql://postgres:postgres@db:5432/jms?schema=public"

  • run prisma generate
    docker-compose exec server sh -c 'yarn run prisma generate'

  • run prisma migration
    docker-compose exec server sh -c 'yarn run prisma migrate dev'

  • run user seeders - seed 5 users in db. This is a needed step for the job service.
    docker-compose exec server sh -c 'yarn run seed:users'
    Reason: personInCharge dropdown is based on users data in DB

  • test out the page and form submission by visiting localhost:3000/job on browser


Notes

  • FE validation should be working
  • required fields are:
    • Customer information
    • Job title
    • job type
    • person in charge
    • other are not required field but please validate according to data type.
  • Tags were set to be an enum - TAG_A, TAG_B, or TAG_C

Screenshot

Uploading chrome_cCjiodNNib.mp4…
chrome_3aTRDonvCj
chrome_xTWhswxESk
image

Nature of task

  • Frontend Integration

Risks

  • [ ]

@jasonchuasun jasonchuasun self-assigned this Sep 12, 2023
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.

2 participants