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

(#380) Add Chocoaltey Coding Livestream #384

Merged
merged 2 commits into from
Feb 14, 2024

Conversation

st3phhays
Copy link
Member

Description Of Changes

The Chocolatey Coding Livestream has been added
to the right side flyout and an alert at the top
of the page.

Motivation and Context

This is a new event we need to let people know about.

Testing

  1. Run the website
  2. Ensure the Chocolatey Coding Livestream is on the home page and inside the right side flyout.
  3. Go to /events, and see the event is the recurring events area.
  4. Go to the event page, and check the contents/dates are correct, and that there is one replay showing at the bottom.
  5. Ensure the Add to Calendar button works.

Change Types Made

  • Bug fix (non-breaking change).
  • Feature / Enhancement (non-breaking change).
  • Breaking change (fix or feature that could cause existing functionality to change).
  • Documentation changes.
  • PowerShell code changes.

Change Checklist

  • Requires a change to the documentation.
  • Documentation has been updated.
  • Tests to cover my changes, have been added.
  • All new and existing tests passed?
  • PowerShell code changes: PowerShell v2 compatibility checked?

Related Issue

@st3phhays st3phhays force-pushed the coding-livestream-event branch from a1b250c to 6c34284 Compare February 13, 2024 18:22
@st3phhays st3phhays self-assigned this Feb 13, 2024
@st3phhays st3phhays requested a review from gep13 February 13, 2024 19:33
@st3phhays st3phhays marked this pull request as ready for review February 13, 2024 19:36
<a href="https://chocolatey.org/events/chocolatey-coding-livestream" rel="noreferrer" target="_blank">
<img class="border rounded mb-3" src="https://img.chocolatey.org/events/01-21-2.jpg" alt="Chocolatey Coding Livestream" />
</a>
<p class="text-start">Join us for the Chocolatey Coding Livestream, where members of our team dive into the heart of open source development by coding live on various Chocolatey projects. Tune in to witness real-time coding, ask questions, and gain insights into the world of package management. Don’t miss this opportunity to engage with our team and contribute to the future of Chocolatey!</p>
Copy link
Member

Choose a reason for hiding this comment

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

@st3phhays I am going to push a slight change to this PR, as there is a gremlin character here, which I believe is causing this artifact on the site:
image

Suggested change
<p class="text-start">Join us for the Chocolatey Coding Livestream, where members of our team dive into the heart of open source development by coding live on various Chocolatey projects. Tune in to witness real-time coding, ask questions, and gain insights into the world of package management. Dont miss this opportunity to engage with our team and contribute to the future of Chocolatey!</p>
<p class="text-start">Join us for the Chocolatey Coding Livestream, where members of our team dive into the heart of open source development by coding live on various Chocolatey projects. Tune in to witness real-time coding, ask questions, and gain insights into the world of package management. Don't miss this opportunity to engage with our team and contribute to the future of Chocolatey!</p>

The Chocolatey Coding Livestream has been added
to the right side flyout and an alert at the top
of the page.
@gep13 gep13 force-pushed the coding-livestream-event branch from 6c34284 to 4494b91 Compare February 14, 2024 09:03
Copy link
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

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

LGTM!

@gep13 gep13 merged commit 26b7442 into chocolatey:main Feb 14, 2024
2 checks passed
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