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 links to the countries #97

Merged
merged 2 commits into from
Oct 26, 2023
Merged

Added links to the countries #97

merged 2 commits into from
Oct 26, 2023

Conversation

manan18
Copy link
Contributor

@manan18 manan18 commented Oct 26, 2023

No description provided.

@manan18
Copy link
Contributor Author

manan18 commented Oct 26, 2023

Added Wikipedia links to all the countries:
Screenshot 2023-10-26 165254

@Virtual4087
Copy link
Owner

Great work @manan18. Thanks for contributing.

Copy link
Owner

@Virtual4087 Virtual4087 left a comment

Choose a reason for hiding this comment

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

@manan18 I think you forgot to add target="_blank" in the links. Do that and we're good to go.

@manan18
Copy link
Contributor Author

manan18 commented Oct 26, 2023

Sure, I'll do that.

@manan18
Copy link
Contributor Author

manan18 commented Oct 26, 2023

@Virtual4087 Please have a look again.

@Virtual4087 Virtual4087 merged commit 6b6990d into Virtual4087:main Oct 26, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants