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

Make all off-site links open in a new tab #577

Merged
merged 10 commits into from Oct 2, 2022
Merged

Make all off-site links open in a new tab #577

merged 10 commits into from Oct 2, 2022

Conversation

letsintegreat
Copy link
Contributor

Thanks for contributing to the Hacktoberfest Swag List 😃 🎉! Before submitting your pull request, please check off as many of the items below as you can:

  1. I have read the Contributing.md file and formatted this PR correctly
  2. I'm not adding a company from the blocklist
  3. I make sure to fix things promptly if an error or suggestion comes up

Thanks and Happy Hacktoberfest! 🎉
Tagging @crweiner to take a look. 👀

Closes #563

1. Be sure you are looking to add something of substance to this project, not just spam PRs. PRs must meet the [Hacktoberfest Quality Standards](https://hacktoberfest.com/participation/).
2. Verify that you have read the [home page](index.md), and [Readme.md on GitHub](https://github.com/crweiner/hacktoberfest-swag-list/blob/master/README.md). You understand that this project is to connect maintainers with developers, not a way to get all the swag you can.
1. Be sure you are looking to add something of substance to this project, not just spam PRs. PRs must meet the [Hacktoberfest Quality Standards](https://hacktoberfest.com/participation/){:target="\_blank"}.
2. Verify that you have read the [home page](index.md), and [Readme.md on GitHub](https://github.com/crweiner/hacktoberfest-swag-list/blob/master/README.md){:target="\_blank"}. You understand that this project is to connect maintainers with developers, not a way to get all the swag you can.
Copy link

Choose a reason for hiding this comment

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

Ordered list item prefix

Update mkdocs.yml enables Attributes List
@ghost
Copy link

ghost commented Oct 1, 2022

👇 Click on the image for a new way to code review
  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

@netlify
Copy link

netlify bot commented Oct 1, 2022

Deploy Preview for hacktoberfest-swag-list ready!

Name Link
🔨 Latest commit 65f2b03
🔍 Latest deploy log https://app.netlify.com/sites/hacktoberfest-swag-list/deploys/633907665c7f1d0008645d83
😎 Deploy Preview https://deploy-preview-577--hacktoberfest-swag-list.netlify.app/contributing
📱 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 settings.

@crweiner
Copy link
Owner

crweiner commented Oct 1, 2022

Sorry for throwing a monkey wrench in some of your plans here, @letsintegreat. 😅 I've merged some PRs so the list is updated. Would you able to update your fork and/or rebase? I really like how you went through and made sure each link opens in a new tab, which I would like to get merged!

@letsintegreat
Copy link
Contributor Author

@crweiner
Ah sure, give me some time while I update the fork and make changes!

@letsintegreat
Copy link
Contributor Author

@crweiner There you have it!

@codeclimate
Copy link

codeclimate bot commented Oct 2, 2022

Code Climate has analyzed commit 65f2b03 and detected 1 issue on this pull request.

Here's the issue category breakdown:

Category Count
Style 1

View more on Code Climate.

@crweiner
Copy link
Owner

crweiner commented Oct 2, 2022

Another beautiful job, thank you again @letsintegreat! This was very helpful, now people can stay on the Swag List and still check out the organizations. I appreciate it! 🎉

great job

@crweiner crweiner added the hacktoberfest-accepted Approved for Hacktoberfest label Oct 2, 2022
@crweiner crweiner merged commit de51aca into crweiner:master Oct 2, 2022
@letsintegreat letsintegreat deleted the external-links branch October 2, 2022 05:05
@letsintegreat
Copy link
Contributor Author

@crweiner Umm, for some reasons your repository has been excluded from hacktoberfest. I am not sure why.

image

Do you know anything about it?

@crweiner
Copy link
Owner

crweiner commented Oct 2, 2022

Thanks for the heads up, I'll reach out to the Hacktoberfest folks to see. 👍

@letsintegreat
Copy link
Contributor Author

@crweiner Alright, looking forward to hearing about any updates.

@crweiner
Copy link
Owner

crweiner commented Oct 5, 2022

Hey @letsintegreat can you please take another look? I reached out to the Hacktoberfest team and they let me know they lifted the exclusion. 😺

@letsintegreat
Copy link
Contributor Author

@crweiner Neat! Excluded status has been removed from your repo. Thanks :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Approved for Hacktoberfest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Make all off-site links open in a new tab
3 participants