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

Form UI Additions #551

Merged
merged 18 commits into from
May 18, 2017
Merged

Form UI Additions #551

merged 18 commits into from
May 18, 2017

Conversation

brittainhard
Copy link
Contributor

@brittainhard brittainhard commented May 8, 2017

Closes #520
Closes #504
Closes #417
Closes #554
Closes #555

@brittainhard
Copy link
Contributor Author

@MattHJensen if its okay with you im going to merge this and push it to master when we get the taxcalc update working.

@MattHJensen
Copy link
Contributor

@brittainhard, are all of these on the test app now? What's the reason for waiting for the taxcalc update?

@brittainhard
Copy link
Contributor Author

@MattHJensen this branch is up on the test app now. I think we can update the main app without issues.

@MattHJensen
Copy link
Contributor

@brittainhard, it looks like taxcalc 0.8.4 is on the test app. I thought we couldn't put 0.8.4 up because of the dropq issue?

@brittainhard
Copy link
Contributor Author

@MattHJensen That's correct. We still have some issues that we are working through with taxcalc. The UI elements in this PR are separate from taxcalc, so there should be no problem.

@MattHJensen
Copy link
Contributor

I see. I just tested this PR. Looks good. When we deploy to ospc.org we should use 0.8.3 for both front end and back end. Will deploying to ospc.org require downtime?

@brittainhard
Copy link
Contributor Author

@MattHJensen When we update the front-end it will automatically start using taxcalc 0.8.4. The heroku build is set to to grab the latest packages. As far as I know, the backend will still be at 0.8.2. I can't access the nodes right now, but i'll double-check with Peter.

@brittainhard
Copy link
Contributor Author

brittainhard commented May 18, 2017

Deploying and updating the nodes to 0.8.3 will take some time.

@brittainhard
Copy link
Contributor Author

@MattHJensen just checked with @PeterDSteinberg and he said that the nodes are at taxcalc 0.8.3.

@MattHJensen
Copy link
Contributor

just checked with @PeterDSteinberg and he said that the nodes are at taxcalc 0.8.3.

That's good.

Could you change something to make the heroku build grab a package you specify? And then I think it would be helpful to have a test that ensures that the nodes and front end are using the same package.

@brittainhard
Copy link
Contributor Author

@MattHJensen the only interface for updating heroku is git. If we wanted to modify the version it uses, we would have to modify the buildpack itself and hard-code it (buildpack is at https://github.com/OpenSourcePolicyCenter/conda-buildpack).

I don't think the UI update will cause errors even with the version being ahead of the nodes. Most of the changes were JavaScript related, and the changes that were made to the django part of the app should not cause errors -- they were database field updates to accommodate the checkbox changes.

@brittainhard brittainhard changed the title Form UI Additions (WIP) Form UI Additions May 18, 2017
@MattHJensen
Copy link
Contributor

MattHJensen commented May 18, 2017 via email

@brittainhard brittainhard merged commit fbd13d4 into master May 18, 2017
@martinholmer martinholmer deleted the bhard/form_ui_additions branch July 19, 2017 01:26
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.

2 participants