Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix iph trough model help links. #1941

Merged
merged 1 commit into from
Dec 4, 2024
Merged

Fix iph trough model help links. #1941

merged 1 commit into from
Dec 4, 2024

Conversation

taylorbrown75
Copy link
Collaborator

Description

The IPH trough model currently links to the IPH MSLF help documents. This pull request fixes the link, to direct to the IPH trough documentation.

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • This change requires a documentation update

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
  • New and existing unit tests pass locally with my changes
  • I have checked my code and corrected any misspellings

@taylorbrown75 taylorbrown75 added csp concentrating solar power UI User interface issue that applies across performance and financial models labels Dec 4, 2024
@taylorbrown75 taylorbrown75 added this to the SAM Fall 2024 Release milestone Dec 4, 2024
@taylorbrown75 taylorbrown75 self-assigned this Dec 4, 2024
@taylorbrown75 taylorbrown75 merged commit 18b331f into develop Dec 4, 2024
8 checks passed
@cpaulgilman cpaulgilman added the added to release notes PR and/or issue has been added to release notes for a public release label Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cpaulgilman cpaulgilman cpaulgilman approved these changes

Assignees

@taylorbrown75 taylorbrown75

Labels
added to release notes PR and/or issue has been added to release notes for a public release csp concentrating solar power UI User interface issue that applies across performance and financial models
Projects
None yet
Milestone
SAM Fall 2024 Release
Development

Successfully merging this pull request may close these issues.
2 participants
@taylorbrown75 @cpaulgilman