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 #45

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Add Social Profile #45

wants to merge 15 commits into from

Conversation

hoangdatngo
Copy link
Contributor

This is the work for this issue: #44.

I have added links to various social profile of some of the speakers in the Fossasia OpenTechSummit Thailand. I also update some of the social profile of the Fossasia Team. I also change linkedin profile of MR. Andrew Lee because it linking to the collabora company.

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

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

Copy link

@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.

@hoangdatngo
Copy link
Contributor Author

@kushthedude Thanks

Copy link

@NishithP2004 NishithP2004 left a comment

Choose a reason for hiding this comment

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

LGTM

@hoangdatngo
Copy link
Contributor Author

@NishithP2004 Thanks

@hoangdatngo
Copy link
Contributor Author

@snitin315 Can you please merge my PR?

index.html Outdated Show resolved Hide resolved
@hoangdatngo
Copy link
Contributor Author

@cweitat I have update index.html file. Can you please review it again?

index.html Outdated
@@ -591,6 +591,12 @@ <h2>Klaikong Vaidhyakarn</h2>
<a target="default" href="https://www.facebook.com/misako.ito">
<i class="icon social_facebook"></i>
</a>
<a target="default" href="https://twitter.com/misako_ito">
Copy link

Choose a reason for hiding this comment

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

This is not even her. Can you check before you even make the changes?

index.html Outdated
@@ -1555,6 +1567,9 @@ <h1>Team</h1>
<a href="https://github.com/mishari" target="default">
<i class="icon fa fa-github"></i>
</a>
<a href="https://www.youtube.com/user/misharim" target="default">
Copy link

Choose a reason for hiding this comment

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

How is this even relevant?

index.html Show resolved Hide resolved
@cweitat
Copy link

cweitat commented Jan 11, 2020

Don't just make a PR for the sake of it. If you don't check before updating, just close this PR.

@hoangdatngo
Copy link
Contributor Author

@cweitat Sorry for that mistake, I haven't check carefully. Can you please review it again. I have check it very careful. If it didn't get merged, I will close this PR. Again sorry for wasting your time.

@hoangdatngo
Copy link
Contributor Author

@snitin315 Can you please review it?

Copy link
Member

@snitin315 snitin315 left a comment

Choose a reason for hiding this comment

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

Squash your commits.

index.html Show resolved Hide resolved
@hoangdatngo
Copy link
Contributor Author

@snitin315 Can you please review it again?

@snitin315
Copy link
Member

@snitin315 Can you please review it again?

To Squash your commits Refer here

@kushthedude
Copy link

You can leave the squashing part.

@hoangdatngo
Copy link
Contributor Author

@kushthedude Can you please review it again?

index.html Outdated
@@ -621,11 +624,14 @@ <h2>Misako Ito</h2>
</a>
<a target="default" href="https://m.facebook.com/putchonguth?fref=nf">
<i class="icon social_facebook"></i>
</a>
<a target="default" href="https://www.youtube.com/user/pxu007">
Copy link

Choose a reason for hiding this comment

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

again, is it relevant?

index.html Outdated
</a>
<a target="default" href="https://sites.google.com/a/ku.th/putchong-uth/home">
<i class="icon fa fa-external-link"></i>
</a>
<a target="default" href="https://www.cpe.ku.ac.th/?page_id=1959">
<a target="default" href="https://www.cpe.ku.ac.th/?page_id=1959">
Copy link

Choose a reason for hiding this comment

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

indent for?

<a href="https://www.linkedin.com/in/jinalfoflia" target="default">
<i class="icon social_linkedin"></i>
</a>
<a target="default" href="https://medium.com/@jinalfoflia">
Copy link

Choose a reason for hiding this comment

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

for what?

Copy link

Choose a reason for hiding this comment

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

why post a link that is old from 3 years ago and not updated?

@@ -1692,7 +1710,10 @@ <h2>Sivalee Anantachart</h2>
</a>
<a target="_self" href="github.com/mariobehling">
<i class="icon fa fa-github"></i>
</a>
</a>
<a href="https://www.youtube.com/user/mariobehling" target="default">
Copy link

Choose a reason for hiding this comment

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

what's the purpose? how is it relevant to the event??

Copy link

Choose a reason for hiding this comment

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

and extra 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.

5 participants