Skip to content

Fix cf energy sales#919

Closed
brtietz wants to merge 1 commit into
patchfrom
fix_cf_energy_sales
Closed

Fix cf energy sales#919
brtietz wants to merge 1 commit into
patchfrom
fix_cf_energy_sales

Conversation

@brtietz

@brtietz brtietz commented Feb 7, 2022

Copy link
Copy Markdown
Collaborator

Cashflow.lk changes to go with NatLabRockies/ssc#750

Additional changes may be needed for "send to excel with equations" please advise

@brtietz brtietz added bug financial Financial model labels Feb 7, 2022
@brtietz brtietz added this to the 2021.12.02 Patch 1 milestone Feb 7, 2022
@brtietz brtietz requested a review from cpaulgilman February 7, 2022 21:01
@cpaulgilman

Copy link
Copy Markdown
Collaborator

@brtietz This should be branched off of Patch instead of Develop. Similar to SSC, my om_cost_inputs branch has other fixes related to cash flow, so I would like to merge fix_cf_energy_sales into om_cost_inputs and then into Patch if that's OK by you.

@brtietz brtietz changed the base branch from develop to patch February 7, 2022 22:15
@brtietz

brtietz commented Feb 7, 2022

Copy link
Copy Markdown
Collaborator Author

@cpaulgilman looks like it was just a target branch issue in the pull request - I've fixed that (I wonder if we should adjust that default during patch season). Yes, feel free to do that merging route.

@cpaulgilman

Copy link
Copy Markdown
Collaborator

Send-to-Excel with equations will be handled when I address #694

@brtietz

brtietz commented Feb 10, 2022

Copy link
Copy Markdown
Collaborator Author

Included in #927 - closing this PR

@brtietz brtietz closed this Feb 10, 2022
@cpaulgilman cpaulgilman added the added to release notes PR and/or issue has been added to release notes for a public release label Feb 21, 2022
@brtietz brtietz deleted the fix_cf_energy_sales branch November 18, 2022 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

added to release notes PR and/or issue has been added to release notes for a public release bug financial Financial model

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants