-
Notifications
You must be signed in to change notification settings - Fork 6.6k
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
feat:Adding Country and State Feilds In Employee Doctype #41379
base: develop
Are you sure you want to change the base?
Conversation
"options": "Country", | ||
"reqd": 1 | ||
}, | ||
{ | ||
"fieldname": "state", | ||
"fieldtype": "Data", | ||
"label": "State", | ||
"mandatory_depends_on": "eval:doc.country==\"United States\"" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"options": "Country", | |
"reqd": 1 | |
}, | |
{ | |
"fieldname": "state", | |
"fieldtype": "Data", | |
"label": "State", | |
"mandatory_depends_on": "eval:doc.country==\"United States\"" | |
"options": "Country" | |
}, | |
{ | |
"fieldname": "state", | |
"fieldtype": "Data", | |
"label": "State" |
Remove these reqd checks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#41392
mam, New PR by removing the reqd checks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Push it to this branch itself?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mam, Pushed to the Same Branch.
This pull request has been automatically marked as inactive because it has not had recent activity. It will be closed within 3 days if no further activity occurs, but it only takes a comment to keep a contribution alive :) Also, even if it is closed, you can always reopen the PR when you're ready. Thank you for contributing. |
I am Working on a US-Payroll App. It Requires a Country and State for Every Employee.
I have added these fields in Employee Doctype in the Address and Contacts Tab and the Address Section. The country is the Link Field to Country Doctype and the state is the Data Field. If the country is equal to the United States, the state will be mandatory as Taxes are calculated based on the state for the "United States" Country