Skip to content
This repository has been archived by the owner on Sep 7, 2024. It is now read-only.

Webinars #792

Merged
merged 2 commits into from
Sep 23, 2023
Merged

Webinars #792

merged 2 commits into from
Sep 23, 2023

Conversation

Susmita-Dey
Copy link
Contributor

@Susmita-Dey Susmita-Dey commented Sep 23, 2023

Fixes Issue

This PR fixes the following issues:

closes #issue-number

Changes proposed

Here comes all the changes proposed through this PR

Check List (Check all the boxes which are applicable)

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • All new and existing tests passed.
  • This PR does not contain plagiarized content.
  • The title of my pull request is a short description of the requested changes.

Screenshots

Add all the screenshots which support your changes

Summary by CodeRabbit

"Chore: Updated Newsletter Hosting Service"

  • The source URLs for iframes in the Register and NewsLetter components across various pages have been updated.
  • The newsletter content is now loaded from https://wemakedevs-newsletter.netlify.app/ instead of https://wemakedevs-newsletter.vercel.app/.
  • This change affects the IndividualWebinar, RoadLessTravelled, ParticularMeetupPage, and PerticularhackathonPage components.
  • Users should not notice any difference as this is a backend infrastructure change.

@coderabbitai
Copy link

coderabbitai bot commented Sep 23, 2023

Walkthrough

The changes across these files primarily involve updating the source URL of various iframes. The original URLs, hosted on Vercel, have been replaced with new URLs hosted on Netlify. This suggests a shift in hosting platforms for the newsletter content.

Changes

File Path Summary
src/components/PageComp/.../Register.jsx Updated iframe source URL from Vercel to Netlify.
src/components/PageComp/RoadLessTravelled/Register.jsx Changed iframe source URL to Netlify.
src/components/layout/JoinUs/NewsLetter.jsx Modified iframe source URL to point to Netlify.
src/pages/ParticularMeetupPage.jsx Replaced iframe source URL with Netlify link.
src/pages/PerticularhackathonPage.jsx Switched iframe source URL to Netlify.

🐇💻

In the land of code, where the brackets lie,

A change was made, under the digital sky.

From Vercel to Netlify, the newsletters flew,

Celebrating the change, says this coding bunny to you! 🎉🥳


Tips

Chat with CodeRabbit Bot (@coderabbitai)

  • Mention @coderabbitai in any review comment for bot assistance.
  • Note: Review comments are made on code diffs or files, not on the PR overview.

Pause Incremental Reviews

  • Insert @coderabbitai: ignore in the PR description to halt the bot's ongoing reviews. Remove the line to resume.

@Susmita-Dey Susmita-Dey merged commit 154aa66 into WeMakeDevs:main Sep 23, 2023
5 checks passed
@github-actions github-actions bot added the team label Sep 23, 2023
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Review Status

Actionable comments generated: 1

Commits Files that changed from the base of the PR and between baf46c0 and fe22b4f.
Files selected for processing (5)
  • src/components/PageComp/IndividualWebinar/Register.jsx (1 hunks)
  • src/components/PageComp/RoadLessTravelled/Register.jsx (1 hunks)
  • src/components/layout/JoinUs/NewsLetter.jsx (1 hunks)
  • src/pages/ParticularMeetupPage.jsx (1 hunks)
  • src/pages/PerticularhackathonPage.jsx (1 hunks)
Files skipped from review due to trivial changes (4)
  • src/components/PageComp/IndividualWebinar/Register.jsx
  • src/components/PageComp/RoadLessTravelled/Register.jsx
  • src/pages/ParticularMeetupPage.jsx
  • src/pages/PerticularhackathonPage.jsx
Additional comments (Suppressed): 1
src/components/layout/JoinUs/NewsLetter.jsx (1)
  • 26-26: Ensure that the new URL https://wemakedevs-newsletter.netlify.app/ is correctly set up and serving the expected content. This change implies a migration from Vercel to Netlify for hosting the newsletter, so it's crucial to verify that the new host is functioning as expected.

src/components/layout/JoinUs/NewsLetter.jsx Show resolved Hide resolved
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant