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

DTT (Staging > Test) [robo-dtt] #44855

Merged
merged 37 commits into from
Feb 15, 2022
Merged

DTT (Staging > Test) [robo-dtt] #44855

merged 37 commits into from
Feb 15, 2022

Conversation

deploy-code-org
Copy link
Contributor

No description provided.

hannahbergam and others added 30 commits February 1, 2022 16:10
Rails 6 changed the way ActionView expects to be initialized. Within Pegasus, we are using a modified version of ActionView to render templates and so that modification needs to be updated.

The nature of our modification includes storing a reference to Sinatra within the instance of ActionView that we initialize; our original implementation simply added a new required initialization variable but the new Rails 6 initialization logic makes that more difficult. So instead, we manually set the instance variable after initialization. This is obviously much clumsier than the original approach, but should be easier to maintain.

The increased complexity of our new initialization logic is all captured in a new factory method.
deploy-code-org and others added 7 commits February 15, 2022 20:00
DTS (Levelbuilder > Staging) [robo-dts]
Add Rails 6 Compatibility to ActionViewSinatra
Remove status from form data after status gets set
make certificate share page responsive, and add call-to-action
X button in Finish Dialog for Dance closes modal instead of redirecting
@deploy-code-org deploy-code-org requested a review from a team as a code owner February 15, 2022 22:16
@deploy-code-org deploy-code-org merged commit 34e19d9 into test Feb 15, 2022
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.

None yet

6 participants