Repository navigation
Allow tax_status to be updateable by consortium admin for consortium and consortium organizations - #979
Conversation
…eate/delete for consortium_admin.
There was a problem hiding this comment.
I’ve taken a look at this and the Tax Status field now appears for both Consortium Leads and Consortium Organizations when updating a Consortium Organization account.
However, the field "Salesforce ID" now appears for Consortium Leads. This should only be visible to Admin. Would it be possible to suppress it for Consortium Leads?
Another small change that would be good to have is the field order. Could we have "Tax Status" appear directly below "Country" for both account types?
…tion. Move tax status field to below country field on create/update organization
|
@svogt0511 On the latest preview here: https://bracco-65r5otp80-datacite.vercel.app/ I logged in as a Consortium and went to update a Consortium Organization. There was a new field that appeared, "Consortium", which provides a way to select the Consortium that the Consortium Organization is part of:
This should not be visible from the Consortium view, since a Consortium shouldn't be able to move their Consortium Organizations to a different Consortium. |
In staging both fields member type and consortium are gone. I will put it back to that. |
|
@KellyStathis - Changes have been made. See the preview: https://bracco-mbhq9o91h-datacite.vercel.app/ Let me know. Thanks again! |


Purpose
closes: #938
Ensure that the "tax status" form entry shows up on the appropriate forms, and is updateable by a consortium admin.
preview: https://bracco-mbhq9o91h-datacite.vercel.app/
Approach
Open Questions and Pre-Merge TODOs
Learning
Types of changes
Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)
Reviewer, please remember our guidelines: