Skip to content

Updates to new ITC-depreciation workflow#2220

Merged
mjprilliman merged 4 commits into
developfrom
itctesting
Jun 26, 2026
Merged

Updates to new ITC-depreciation workflow#2220
mjprilliman merged 4 commits into
developfrom
itctesting

Conversation

@janinefreeman

@janinefreeman janinefreeman commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Fixes typo in one of the option selections and adds units to various UI objects that are missing units.

@mjprilliman and @brtietz Is this a bug? I would expect ITC qualifying to be 0 based on the allocations that are set up? Or, if these calculated boxes are no longer relevant with the second radio button selection, they should be greyed out or disappear.
image

@mjprilliman

Copy link
Copy Markdown
Collaborator

Fixes typo in one of the option selections and adds units to various UI objects that are missing units.

@mjprilliman and @brtietz Is this a bug? I would expect ITC qualifying to be 0 based on the allocations that are set up? Or, if these calculated boxes are no longer relevant with the second radio button selection, they should be greyed out or disappear. image

I vote grey out when not in use. I'm not familiar enough with the ssc changes to know if we can zero out the values when not in use, zeroing out also loses people's selections if they go back to the first option. @brtietz would have to weigh in on that part.

@janinefreeman

Copy link
Copy Markdown
Collaborator Author

More specifically, the equation for those calculated values could be conditionally calculated from the selections in the "depreciation class allocations and bonus depreciation" section, if that radio button option is selected, versus being calculated from the inputs above. I'm not suggesting to zero them out.

@mjprilliman
mjprilliman merged commit 7622031 into develop Jun 26, 2026
6 checks passed
@mjprilliman
mjprilliman deleted the itctesting branch June 26, 2026 15:06
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