CDLUC3 / dmptool

DMPTool version of the DMPRoadmap codebase
https://dmptool.org
MIT License
57 stars 13 forks source link

Rails 5 UAT issues #214

Closed mariapraetzellis closed 3 years ago

mariapraetzellis commented 3 years ago

Listing out all issues identified while doing Rails 5 testing on dmptool-stg

briri commented 3 years ago

the ID that gets placed in there is the unqiue id of the record in the DB. It corresponds to the one you see in the URL. We can comment out this behavior for DMPTool if you'd like

briri commented 3 years ago

The affiliation change issue stemmed from the fact that some of the Orgs that have been auto-created by the new RORified Org selector were created with a blank language_id. The new Rails 5 code sets this value to the default language if none is specified so it should not be an issue going forward.

I am adding an upgrade script that will fix any of these records when we release Rails5. I have run it already on stage so you should now be able to change your affiliation.'

The same scenario was causing the issue when enabling feedback. I added as script to default those as well.

briri commented 3 years ago

The ORCID issue was a configuration error. It is now connected with the ORCID sandbox (from the stage server)

briri commented 3 years ago

I'm not able to replicate the "Editing a template by re-ordering questions gives a 500" issue @mariapraetzellis. If there are 2 questions numbered 1 and 2 and I try to update 2 to be 1, I get an error message that the number must be unique. Is that what you mean?

I could also not replicate ""Error" message displayed when selecting the option on a template that should have triggered a notification." I added a conditional to the UCOP template (2nd question in 1st section is triggered when you select 'Maybe') and it sent the email.

mariapraetzellis commented 3 years ago

Confirmed that I am able to add a contributor to a plan successfully.

I also successfully re-ordered questions on a template.

briri commented 3 years ago

I think all of these issues have been resolved