Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

feat: indicate required fields in new Related Person Modal #1950

Merged
merged 2 commits into from
Mar 29, 2020

Conversation

jamesinsf-git
Copy link
Contributor

Fixes #1927 .

Changes proposed in this pull request:

  • add isRequired to Person Type field
  • add isRequired to Relationship Type field

Newly added dependencies with Bundlephobia links:

N/A

@jsf-clabot
Copy link

jsf-clabot commented Mar 29, 2020

CLA assistant check
All committers have signed the CLA.

@vercel
Copy link

vercel bot commented Mar 29, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/hospitalrun/hospitalrun-frontend/4jcqvmeuu
✅ Preview: https://hospitalrun-frontend-git-fork-jamesinsf-git-firstissue.hospitalrun.now.sh

@jackcmeyer jackcmeyer changed the title fix: indicate required fields in new Related Person Modal feat: indicate required fields in new Related Person Modal Mar 29, 2020
@jackcmeyer jackcmeyer merged commit e17dd1d into HospitalRun:master Mar 29, 2020
@jamesinsf-git jamesinsf-git deleted the firstIssue branch March 30, 2020 23:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New Related Persons Modal should mark required fields
3 participants