Skip to content

Check for standalone battery before loading charging priority variables#1255

Merged
mjprilliman merged 2 commits into
developfrom
standalone_batt_priority_exclusion
Nov 14, 2022
Merged

Check for standalone battery before loading charging priority variables#1255
mjprilliman merged 2 commits into
developfrom
standalone_batt_priority_exclusion

Conversation

@mjprilliman

Copy link
Copy Markdown
Collaborator

Description

Do not show battery charging priority inputs on manual dispatch page when battery is standalone

Fixes #1254

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

If you have added a new compute module in a SSC pull request related to this one, be sure to check the Process Requirements.

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

@brtietz brtietz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing!

@mjprilliman mjprilliman merged commit 7bbc400 into develop Nov 14, 2022
@mjprilliman mjprilliman deleted the standalone_batt_priority_exclusion branch November 14, 2022 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BTM battery charging priority breaking BTM standalone battery

2 participants