Skip to content

Add VanPy and VanJS to website #30

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 3 commits into from
Jan 18, 2019
Merged

Conversation

Qard
Copy link
Member

@Qard Qard commented Jan 10, 2019

Updating the website to include VanPy and VanJS.

@Qard Qard added the enhancement New feature or request label Jan 10, 2019
@Qard Qard self-assigned this Jan 10, 2019
</li>
<li className="style3">
<span className="icon fa-signal"></span>
<strong>7,931</strong> members
<strong>12,000+</strong> members
Copy link
Collaborator

Choose a reason for hiding this comment

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

Don't actually do this if you've stepped away or something,

but we could just make NUM_MEMBERS a variable if you want the two to match easily ;)

@@ -59,7 +59,7 @@ class Index extends React.Component {

<section id="first" className="main special">
<header className="major">
<h2>founding events</h2>
<h2>our events</h2>
Copy link
Collaborator

Choose a reason for hiding this comment

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

holy github diff

Copy link
Member

Choose a reason for hiding this comment

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

😄

Copy link
Member

@keywordnew keywordnew left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you for this PR. Very much needed! 🎉

@keywordnew
Copy link
Member

Merging, folks!

@keywordnew keywordnew merged commit ba9fa30 into development Jan 18, 2019
@Qard Qard deleted the add-vanjs-and-vanpy branch January 20, 2019 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants