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.

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

[📈] Added Accenture, rejigged Sponsor UX & New PR Checl #39

Merged
merged 6 commits into from
Oct 16, 2024
Merged

Conversation

RJMurg
Copy link
Member

@RJMurg RJMurg commented Oct 16, 2024

Added Accenture as a sponsor.
Added sponsor tier text.
Put sponsors in a card,
Added a check that test builds the container to ensure that there are no bugs. (Step towards #13)

UX Example:
image

This closes #37
This opens #36
This opens #38

@RJMurg RJMurg changed the title [📈] Added Accenture as Sponsor & rejigged look of Sponsors [📈] Added Accenture, rejigged Sponsor UX & New PR Checl Oct 16, 2024
@RJMurg RJMurg added bug Something isn't working enhancement New feature or request labels Oct 16, 2024
Copy link
Member

@ArcaneIRE ArcaneIRE left a comment

Choose a reason for hiding this comment

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

The cards could do with a little more fine tuning but this is a good start!

src/site/assets/img/sponsors/accenture.png Show resolved Hide resolved
@RJMurg
Copy link
Member Author

RJMurg commented Oct 16, 2024

New UX:
image

@RJMurg RJMurg requested a review from ArcaneIRE October 16, 2024 21:48
@RJMurg RJMurg merged commit 168d3de into main Oct 16, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Container did not properly deploy due to typo in Nunjuck template
2 participants