Skip to content

Commit

Permalink
Update display_name when category is changed (#195)
Browse files Browse the repository at this point in the history
for ALExpenseList. Additionally, removes `complete_attribute="exists"`
from the income demo expense list, as it's not needed, and can cause
issues with review screens.

This is an upstream fix for an issue also fixed in
SuffolkLITLab/docassemble-ALAffidavitOfIndigency#49.
  • Loading branch information
BryceStevenWilley authored Sep 28, 2023
1 parent ddd0d16 commit d9a3648
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions docassemble/ALToolbox/data/questions/al_income.yml
Original file line number Diff line number Diff line change
Expand Up @@ -544,6 +544,8 @@ fields:
default: 12
code: |
times_per_year_for_expenses
validation code: |
x[i].display_name = expense_terms.get(x[i].source, x[i].source)
---
generic object: ALExpenseList
need:
Expand Down
2 changes: 1 addition & 1 deletion docassemble/ALToolbox/data/questions/al_income_demo.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ objects:
- al_value_list: ALSimpleValueList.using(
complete_attribute='complete',
ask_number=True)
- al_expense_list: ALExpenseList.using(auto_gather=False, complete_attribute="exists")
- al_expense_list: ALExpenseList.using(auto_gather=False)
---
id: interview order
mandatory: True
Expand Down

0 comments on commit d9a3648

Please sign in to comment.