Skip to content

Conversation

@hdunham
Copy link
Collaborator

@hdunham hdunham commented Sep 30, 2025

Still TODO, some can probably wait til after FY end:

  • add sector inputs to all inputs test
  • update CHANGELOG
  • add try catches to sector defaults functions to make more robust
  • expand documentation

Please check if the PR fulfills these requirements

  • CHANGELOG.md is updated
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • Any new Django model inputs have also been added to job/test/posts/all_inputs_test.json

What kind of change does this PR introduce?

feature

What is the current behavior?

1 set of defaults based on commercial/industrial analyses

What is the new behavior (if this is a feature change)?

defaults change depending on user specified sector (and additional details if sector=federal)

Does this PR introduce a breaking change?

no

Other information:

see #662 for comments

@hdunham hdunham merged commit e2240bd into develop Oct 6, 2025
1 check passed
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.

3 participants