Skip to content

Update NREL urls to new NLR versions of the urls#711

Open
hdunham wants to merge 10 commits into
developfrom
nlr-urls
Open

Update NREL urls to new NLR versions of the urls#711
hdunham wants to merge 10 commits into
developfrom
nlr-urls

Conversation

@hdunham
Copy link
Copy Markdown
Collaborator

@hdunham hdunham commented Mar 6, 2026

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?

Patch

What is the current behavior?

Using/listing old NREL URLs that will be discontinued at the end of May (doesn't include github.nrel.gov)

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

Update to new NLR versions of the URLs

Does this PR introduce a breaking change?

no

Other information:

@hdunham hdunham marked this pull request as draft March 6, 2026 17:58
@hdunham hdunham requested review from Bill-Becker and adfarth May 14, 2026 20:25
@hdunham hdunham marked this pull request as ready for review May 14, 2026 20:26
Copy link
Copy Markdown
Collaborator

@Bill-Becker Bill-Becker left a comment

Choose a reason for hiding this comment

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

Looks good to me! But I suggest we test deploying to the development or staging servers on this branch first, to make sure it doesn't break deployments. And future work, update the licensing language for "Alliance for Sustainable Energy" -> "Alliance for Energy Innovation". But we'll want to confirm with our licensing team on that.

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