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

[PCC-1176] Starter kit redesign #297

Merged
merged 57 commits into from
Sep 3, 2024
Merged

[PCC-1176] Starter kit redesign #297

merged 57 commits into from
Sep 3, 2024

Conversation

a11rew
Copy link
Collaborator

@a11rew a11rew commented Aug 20, 2024

No description provided.

a11rew and others added 23 commits August 13, 2024 14:02
@a11rew
feat: add assets
aa00787
@a11rew
feat: add manypkg for dependency conflict management
3c48258
@a11rew
feat: add design system button
8f674e0
@a11rew
feat(next-starter-ts): add updated header
1527722
@a11rew
feat: add pantheon logos
4e6c2a5
@a11rew
feat: use poppins font
372b057
@a11rew
Merge branch 'ag/pcc-1527/starter-kit-redesign-design-system' of gith…
002beb7 
…ub.com:pantheon-systems/pantheon-content-cloud-sdk into ag/pcc-1528/starter-kit-redesign-header-footer
@a11rew
Merge pull request #293 from pantheon-systems/ag/pcc-1527/starter-kit…
db6f5f4 
…-redesign-design-system

[PCC-1527] Design system elements implementation
@a11rew
feat: add header and header nav menu
6d1682e
@a11rew
fix: re-add image
b9682ca
@a11rew
Merge branch 'ag/pcc-1176/starter-kit-redesign' of github.com:pantheo…
9392aa3 
…n-systems/pantheon-content-cloud-sdk into ag/pcc-1528/starter-kit-redesign-header-footer
@a11rew
feat: add header to other js and app-router starters
91ee28b
@a11rew
feat: add updated footer
64cd872
@a11rew
Merge pull request #294 from pantheon-systems/ag/pcc-1528/starter-kit…
b12b581 
…-redesign-header-footer

[PCC 1528] Redesigned header and footer
@aumkar
Od/pcc 1536/article blog (#296)
d89aac5
@a11rew
wip: grid page
ae3a2b3
@a11rew
wip: grid layout
7e36aa2
@a11rew
feat: add error pages
aae01cf
@a11rew
Merge pull request #299 from pantheon-systems/ag/pcc-1539/error-pages
9631e62 
[PCC-1539] Error pages
@a11rew
fix: use css instead of deprecated objectFit property
61434f7
@a11rew
feat: add media preview smart component
91286b5
@a11rew
feat: add error boundaries to smart components
bece82a
@a11rew
Merge pull request #300 from pantheon-systems/ag/pcc-1531/smart-compo…
785530c 
…nents

[PCC-1531] Redesigned smart components
@guardrails GuardRails
Copy link

guardrails bot commented Aug 21, 2024 •
edited
Loading

⚠️ We detected 9 security issues in this pull request:

Insecure Processing of Data (9)
Severity Details Docs
Medium Title: Unescaped user input in React dangerouslySetInnerHTML

pantheon-content-cloud-sdk/starters/nextjs-starter-approuter-ts/components/smart-components/media-preview/providers.tsx

Line 191 in 62e85b1

dangerouslySetInnerHTML={{ __html: data.html }}
📚
Medium Title: Unescaped user input in React dangerouslySetInnerHTML

pantheon-content-cloud-sdk/starters/nextjs-starter/components/smart-components/media-preview/providers.jsx

Line 116 in 62e85b1

dangerouslySetInnerHTML={{ __html: html }}
📚
Medium Title: Unescaped user input in React dangerouslySetInnerHTML

pantheon-content-cloud-sdk/starters/nextjs-starter/components/smart-components/media-preview/providers.jsx

Line 139 in 62e85b1

dangerouslySetInnerHTML={{ __html: data.html }}
📚
Medium Title: Unescaped user input in React dangerouslySetInnerHTML

pantheon-content-cloud-sdk/starters/nextjs-starter/components/smart-components/media-preview/providers.jsx

Line 184 in 62e85b1

dangerouslySetInnerHTML={{ __html: data.html }}
📚
Medium Title: Unescaped user input in React dangerouslySetInnerHTML

pantheon-content-cloud-sdk/starters/nextjs-starter-ts/components/smart-components/media-preview/providers.tsx

Line 127 in 62e85b1

dangerouslySetInnerHTML={{ __html: html }}
📚
Medium Title: Unescaped user input in React dangerouslySetInnerHTML

pantheon-content-cloud-sdk/starters/nextjs-starter-ts/components/smart-components/media-preview/providers.tsx

Line 150 in 62e85b1

dangerouslySetInnerHTML={{ __html: data.html }}
📚
Medium Title: Unescaped user input in React dangerouslySetInnerHTML

pantheon-content-cloud-sdk/starters/nextjs-starter-ts/components/smart-components/media-preview/providers.tsx

Line 195 in 62e85b1

dangerouslySetInnerHTML={{ __html: data.html }}
📚
Medium Title: Unescaped user input in React dangerouslySetInnerHTML

pantheon-content-cloud-sdk/starters/nextjs-starter-approuter-ts/components/smart-components/media-preview/providers.tsx

Line 123 in 62e85b1

dangerouslySetInnerHTML={{ __html: html }}
📚
Medium Title: Unescaped user input in React dangerouslySetInnerHTML

pantheon-content-cloud-sdk/starters/nextjs-starter-approuter-ts/components/smart-components/media-preview/providers.tsx

Line 146 in 62e85b1

dangerouslySetInnerHTML={{ __html: data.html }}
📚

More info on how to fix Insecure Processing of Data in JavaScript.

👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

@a11rew a11rew marked this pull request as ready for review August 31, 2024 20:18
@a11rew a11rew requested review from kevinstubbs and aumkar and removed request for kevinstubbs August 31, 2024 20:18
@a11rew a11rew merged commit e9db4dd into main Sep 3, 2024
10 checks passed
@a11rew a11rew deleted the ag/pcc-1176/starter-kit-redesign branch September 3, 2024 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aumkar aumkar Awaiting requested review from aumkar

@kevinstubbs kevinstubbs Awaiting requested review from kevinstubbs

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@a11rew @aumkar