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

Updated DRO Column to exclude the words Development Review Office and… #1888

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

dipitvasdev
Copy link
Member

… added value Select... to the dropdown

Fixes #1650

What changes did you make?

  • Changed the names of the DROs to exclude the words Development Review Office based on Bonnie's feedback, using a new migration
  • Added a value "Select..." as the default or reset value in case the user wants to change the DRO value, this will reflect in backend as null in the project table.
  • Removed some console.log statements from the last commit

Why did you make the changes (we will use this info to test)?

  • Migration was created to update the names and store them alphabetically
  • To add select value, backend code was modified to handle null and update project table accordingly
  • Code cleanup

Issue-Specific User Account

If you registered a new, temporary TDM User Account for this issue, indicate the
username (i.e., email address) for the account.

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

Visuals before changes are applied image
Visuals after changes are applied image

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.

Implement the DRO and Admin Notes columns for Projects
1 participant