Skip to content

DEVDOCS-5796 [new]: Scripts API, add integrity hash#229

Merged
bc-andreadao merged 20 commits intomainfrom
DEVDOCS-5796-pci-src
Apr 29, 2024
Merged

DEVDOCS-5796 [new]: Scripts API, add integrity hash#229
bc-andreadao merged 20 commits intomainfrom
DEVDOCS-5796-pci-src

Conversation

@bc-andreadao
Copy link
Contributor

@bc-andreadao bc-andreadao commented Apr 23, 2024

DEVDOCS-5796

What changed?

  • update Scripts yml file to include hash property
  • update Scripts doc to include info about how BC handles integrity hashes

Release notes draft

  • The newly-released SRI hash feature is now available to use. Now, you’ll be able to add SRI hashes to external SRC scripts.

@bc-andreadao bc-andreadao marked this pull request as ready for review April 23, 2024 17:09
@bc-andreadao bc-andreadao changed the base branch from main to DEVDOCS-5796-base April 23, 2024 17:48
@bc-andreadao bc-andreadao requested a review from bc-erich April 23, 2024 17:54
@bc-andreadao
Copy link
Contributor Author

cc @bigcommerce/merc-team

bc-andreadao and others added 8 commits April 23, 2024 14:04
Co-authored-by: Sarah Riehl <sarah.riehl@bigcommerce.com>
Co-authored-by: Sarah Riehl <sarah.riehl@bigcommerce.com>
Co-authored-by: Sarah Riehl <sarah.riehl@bigcommerce.com>
Co-authored-by: Sarah Riehl <sarah.riehl@bigcommerce.com>
Co-authored-by: Sarah Riehl <sarah.riehl@bigcommerce.com>
Copy link
Contributor

@bc-traciporter bc-traciporter left a comment

Choose a reason for hiding this comment

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

I made some suggestions, you can take or leave them

bc-andreadao and others added 2 commits April 24, 2024 13:42
Co-authored-by: Traci Porter <traci.porter@bigcommerce.com>
Co-authored-by: Traci Porter <traci.porter@bigcommerce.com>
bc-andreadao and others added 2 commits April 24, 2024 13:52
Co-authored-by: Eric Her <40370666+bc-erich@users.noreply.github.com>
Co-authored-by: Eric Her <40370666+bc-erich@users.noreply.github.com>
@bc-andreadao bc-andreadao changed the base branch from DEVDOCS-5796-base to main April 24, 2024 19:02
Copy link
Contributor

@bc-traciporter bc-traciporter left a comment

Choose a reason for hiding this comment

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

LGTM

@bc-andreadao bc-andreadao merged commit 4bcd141 into main Apr 29, 2024
@bc-andreadao bc-andreadao deleted the DEVDOCS-5796-pci-src branch April 29, 2024 17:38
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.

4 participants