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

Add Social Profile #64

Closed
wants to merge 3 commits into from
Closed

Add Social Profile #64

wants to merge 3 commits into from

Conversation

hoangdatngo
Copy link
Contributor

This is the work for this issue: #63

I have added links to various social profile of some of the speakers in the Fossasia OpenTechSummit Vietnam.

The deployment link: https://hoangdatngo.github.io/vn.opentechsummit.asia/

@mariobehling, @kushthedude, @hpdang Can any mentor please review my PR?

@hoangdatngo
Copy link
Contributor Author

@liveHarshit, @kushthedude, @mariobehling Can any mentor please review this. I have been waiting for 2 days.

Copy link
Contributor

@kushthedude kushthedude left a comment

Choose a reason for hiding this comment

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

Looks good

@hoangdatngo
Copy link
Contributor Author

@kushthedude Can you please merge my PR?

@kushthedude
Copy link
Contributor

kushthedude commented Jan 12, 2020 via email

@hoangdatngo
Copy link
Contributor Author

@kushthedude ok, thanks. You review so fast!

@@ -551,7 +555,8 @@ <h1>Speakers</h1>
<a href="https://github.com/rbscholtus"><i class="icon fa fa-github"></i></a>
<a href="https://twitter.com/rbscholtus"><i class="icon fa fa-twitter"></i></a>
<a href="https://www.facebook.com/barend.scholtus"><i class="icon fa fa-facebook"></i></a>
</div>
<a href="https://www.instagram.com/barend.scholtus/"><i class="icon fa fa-instagram"></i></a>
Copy link

Choose a reason for hiding this comment

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

follow the indent

@@ -442,10 +442,13 @@ <h1>Speakers</h1>
<div class="hover-state text-center preserve3d">
<div class="social-links vertical-align">
<a href="https://github.com/mariobehling/"><i class="icon fa fa-github"></i></a>
<a href="https://www.flickr.com/photos/rportrr/"><i class="icon fa fa-flickr"></i></a>
Copy link

Choose a reason for hiding this comment

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

what is this? how is it relevant?

<a href="https://twitter.com/mariobehling" target="default"><i class="icon social_twitter"></i></a>
<a href="https://www.facebook.com/mariobehling" target="default"><i class="icon social_facebook"></i></a>
<a href="https://www.linkedin.com/in/mariobehling" target="_self"><i class="icon social_linkedin"></i></a>
</div>
<a href="https://www.youtube.com/user/mariobehling" target="default"><i class="icon social_youtube"></i></a>
<a href="https://blog.ffii.org/author/mariobehling/" target="_self"><i class="icon fa fa-external-link"></i></a>
Copy link

Choose a reason for hiding this comment

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

how is link that is updated at 2015 useful?

<a href="https://twitter.com/mariobehling" target="default"><i class="icon social_twitter"></i></a>
<a href="https://www.facebook.com/mariobehling" target="default"><i class="icon social_facebook"></i></a>
<a href="https://www.linkedin.com/in/mariobehling" target="_self"><i class="icon social_linkedin"></i></a>
</div>
<a href="https://www.youtube.com/user/mariobehling" target="default"><i class="icon social_youtube"></i></a>
Copy link

Choose a reason for hiding this comment

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

how is the channel useful in telling the profile of person??

Comment on lines +745 to +747
<div class="social-links vertical-align">
<a href="https://www.linkedin.com/in/tantrieuf31/"><i class="icon fa fa-linkedin"></i></a>
</div>
Copy link

Choose a reason for hiding this comment

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

follow the indent

Comment on lines +761 to +763
<div class="social-links vertical-align">
<a href="https://www.linkedin.com/in/trang1/"<i class="icon fa fa-linkedin"></i></a>
</div>
Copy link

Choose a reason for hiding this comment

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

follow the indent

@@ -767,7 +778,8 @@ <h1>Speakers</h1>
<a href="https://twitter.com/felixbecquart"><i class="icon social_twitter"></i></a>
<a href="https://github.com/FelixBecquart1990"><i class="icon fa fa-github"></i></a>
<a href="https://www.facebook.com/felix.becquart"><i class="icon social_facebook"></i></a>
</div>
<a href="https://www.linkedin.com/in/f%C3%A9lix-becquart-8a5a46a6/"><i class="icon social_linkedin"></i></a>
Copy link

Choose a reason for hiding this comment

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

follow the indent

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.

3 participants