Skip to content

Fix #98: Make hyperlinks open in new tab #105

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

Merged
merged 2 commits into from
Sep 10, 2017

Conversation

abarya
Copy link
Contributor

@abarya abarya commented Sep 10, 2017

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.098% when pulling c5d9d74 on abhishekarya286:link-newtab into a4a25fc on OpenSourceHelpCommunity:develop.

@@ -45,7 +45,7 @@ <h3 class="panel-title">{{ session.title }}</h3>
<div class="line-gap-10px"></div>
<i>{{ session.start_date }} - {{ session.end_date }}</i>
<div class="line-gap-10px"></div>
<button type="button" class="btn btn-lg btn-primary sRegister">
<button type="button" class="btn btn-lg btn-primary s">
Copy link
Member

Choose a reason for hiding this comment

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

extraneous change. @abhishekarya286 Please don't change anything that are not relevant to the issue.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry, I did this by mistake. I have edited it now.

Copy link
Member

Choose a reason for hiding this comment

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

@abhishekarya286 It's ok to make mistakes 😉
One learn from his mistakes

@vaibhavsingh97
Copy link
Member

LGTM
Awesome!! @abhishekarya286 Congratulations on your first PR 🎉 🎉. You have done a great job.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.098% when pulling df9859f on abhishekarya286:link-newtab into a4a25fc on OpenSourceHelpCommunity:develop.

@abarya
Copy link
Contributor Author

abarya commented Sep 10, 2017

Thanks @vaibhavsingh97!!

@vaibhavsingh97
Copy link
Member

vaibhavsingh97 commented Sep 10, 2017

@abhishekarya286 You are so fast!! 👍This PR is ready to merge.

@@ -15,7 +15,7 @@
<div class="container">
<h3>Our Mission</h3>
<p class="bi">
<b>"</b>Open Source Help Community is an a virtual platform to help people to start contributing to open source projects<b>"</b>
<b>"</b>Open Source Help Community is a virtual platform to help people to start contributing to open source projects<b>"</b>
Copy link
Contributor

Choose a reason for hiding this comment

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

thanks for this fix too. :)

@nikhita nikhita merged commit f689520 into OpenSourceHelpCommunity:develop Sep 10, 2017
@nikhita
Copy link
Contributor

nikhita commented Sep 10, 2017

@abhishekarya286 Congratulations on getting your first PR merged in OSHC! 😄

@abarya
Copy link
Contributor Author

abarya commented Sep 11, 2017

Thanks @nikhita

@abarya abarya deleted the link-newtab branch September 18, 2017 18:06
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.

4 participants