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

Update sites.js #69

Merged
merged 2 commits into from
Jun 4, 2024
Merged

Update sites.js #69

merged 2 commits into from
Jun 4, 2024

Conversation

elijah0528
Copy link
Contributor

To add your site to the webring, you must be a current student or an alum of the undergraduate Software Engineering program at the University of Waterloo (Waterloo, ON, Canada).

Full Name: Elijah Kurien

Cohort Year: 2028

Full website URL: elijahkurien.com

So we know you're human:

LinkedIn or a similar profile: https://www.linkedin.com/in/elijahkurien/

@github-actions github-actions bot requested review from janakitti and simcard0000 June 4, 2024 07:31
@github-actions github-actions bot added the add site Add a new website to the webring label Jun 4, 2024
Copy link

netlify bot commented Jun 4, 2024

Deploy Preview for eager-mccarthy-ee07e5 ready!

Name Link
🔨 Latest commit 79054e9
🔍 Latest deploy log https://app.netlify.com/sites/eager-mccarthy-ee07e5/deploys/665f428567583100087e5de3
😎 Deploy Preview https://deploy-preview-69--eager-mccarthy-ee07e5.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Owner

@simcard0000 simcard0000 left a comment

Choose a reason for hiding this comment

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

Hey @elijah0528! Thanks for submitting a PR. Two nits:

  • Add your site entry to the end of the allSites array in the sites.js file (instead of the start).
  • Could you link back to the webring somewhere on your site if possible?

@elijah0528
Copy link
Contributor Author

Hey Simran – Made these changes to the sites.js! SE Webring will be pushed with my latest updates to my website where I flesh out each of my experiences. Is this okay?

@simcard0000
Copy link
Owner

@elijah0528 Thanks for the updates! Yeah, as long as you eventually add a link back to the webring somewhere on your site you should be fine.

@simcard0000 simcard0000 self-requested a review June 4, 2024 18:11
@simcard0000 simcard0000 merged commit d9d1464 into simcard0000:main Jun 4, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add site Add a new website to the webring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants