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

Switch domain list in the Dominant Overstory and Dominant Understory fields #122

Closed
ToddVolkening opened this issue Apr 3, 2018 · 7 comments

Comments

@ToddVolkening
Copy link

Habitat > Dominant Overstory and Dominant Understory domain lists are still switched.

Take the domain list that's currently fueling the 'Dominant Overstory' field and instead apply it to the 'Dominant Understory' field. Conversely, take the domain list for 'Dominant Understory' field and apply it to the 'Dominant Overstory' field.

Previously referenced as #75 in Google Word Doc & AGRC GitHub #78

Referenced as UDWR #412

@steveoh
Copy link
Member

steveoh commented Apr 4, 2018

original #78
pr #98

Please list the values that are supposed to be in each domain. This migration should have swapped the values.

@steveoh
Copy link
Member

steveoh commented Apr 5, 2018

@ToddVolkening Please list the values that are supposed to be in each domain. This migration should have swapped the values.

@ToddVolkening
Copy link
Author

ToddVolkening commented Apr 9, 2018 via email

@steveoh steveoh self-assigned this Apr 9, 2018
@steveoh steveoh closed this as completed in ddd65d4 Apr 9, 2018
@stdavis stdavis reopened this Jun 21, 2018
@stdavis
Copy link
Member

stdavis commented Jun 21, 2018

This is wrong AGAIN!

@steveoh
Copy link
Member

steveoh commented Jun 21, 2018

ok. i cannot do it right so I'm not doing it.

@steveoh steveoh removed their assignment Jun 21, 2018
@stdavis
Copy link
Member

stdavis commented Jun 21, 2018

I wonder if this was a mistake with the xml file...

@stdavis
Copy link
Member

stdavis commented Jun 22, 2018

DATABASESCHEMA.XML looks correct to me:
image
image

I bet that I just forgot to run migration_78.py after you fixed it the last time. I went ahead and ran it on the test database and the app reflects the correct values now.

@stdavis stdavis closed this as completed Jun 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants