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

TaxBrain error messages when adjusting standard deduction #828

Closed
siervicul opened this issue Feb 28, 2018 · 3 comments
Closed

TaxBrain error messages when adjusting standard deduction #828

siervicul opened this issue Feb 28, 2018 · 3 comments

Comments

@siervicul
Copy link

Prior to the latest update, TaxBrain used to provide a warning when reducing the standard deduction amounts below the defaults (I understand that this is due to concerns about imputing itemized deduction amounts to tax units that currently take the standard deduction). However, the warning page would keep all amounts as entered by the user, and the user could simply click on the "show me the results" button to proceed with the lower standard deduction amounts.

Now, making any changes (increase or decrease) to the default standard deduction amounts (but not apparently to the additional standard deduction amounts) with the default start year of 2017 produces a 500 internal server error. If you change the start year to 2018 and then try to decrease the standard deduction amounts, TaxBrain provides a warning as before, but instead of retaining the values input by the user, all values on the form are reset to defaults and the start year is changed to 2013. There appears to be no way to proceed past the warning with a reduced standard deduction.

@hdoupe
Copy link
Collaborator

hdoupe commented Feb 28, 2018

@siervicul Thanks for the bug report. PR #827 will resolve this issue. A partial solution is on the test app: http://ospc-taxes7.herokuapp.com/taxbrain/

You have to re-enter the data from your previous run, but then it should work just fine.

We are working to add back the ability to display the user inputs from the original page when showing the warning and error messages. This release changed how the input data is stored and refactored the logic for processing it. Our apologies for missing the errors/warnings case.

@hdoupe
Copy link
Collaborator

hdoupe commented Feb 28, 2018

@siervicul The test app now has the correct behavior, including saving your input from the original page.

@hdoupe
Copy link
Collaborator

hdoupe commented Mar 1, 2018

@siervicul Thanks for opening #828. PolicyBrain 1.4.2 resolves this issue and is running on the normal site (https://www.ospc.org/).

@hdoupe hdoupe closed this as completed Mar 1, 2018
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

No branches or pull requests

2 participants