Skip to content
This repository was archived by the owner on Apr 1, 2021. It is now read-only.

Conversation

@daniel-chung
Copy link
Contributor

Adding a new JS reference article on Assignment Operators

@daniel-chung
Copy link
Contributor Author

This is my first PR on the wiki page--happy to make updates if I'm not conforming to style and/or guidelines.

@ghost ghost self-assigned this Jun 10, 2016
@ghost ghost added QA JavaScript labels Jun 10, 2016
@@ -0,0 +1,75 @@
# Javascript Assignment Operators

Copy link

Choose a reason for hiding this comment

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

Remove this line

@daniel-chung
Copy link
Contributor Author

Thanks for the comments, @atjonathan !
Made all of the updates--but let me know if I missed any.

@@ -0,0 +1,74 @@

## Introduction
Copy link
Member

Choose a reason for hiding this comment

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

Change this to something else like JavaScript Assignment Operators or somethign along the lines. We use this header for the article name in the site's wiki so it is important to have a h1

# JavaScript Assignment Operators

Copy link
Member

Choose a reason for hiding this comment

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

-## Introduction
+### Introduction

This is so it will display data up to the firs ## in the chat, as it is right now, calling that article from the chat would only give the title and no data snippet.

@daniel-chung
Copy link
Contributor Author

Thanks for the feedback, @Rafase282 . I just added the title back

@Rafase282
Copy link
Member

@daniel-chung One last thing, fix that and I'll merge. Thanks for your contribution.

@daniel-chung
Copy link
Contributor Author

@Rafase282 done. Thanks again!

@Rafase282 Rafase282 merged commit 5f87478 into freeCodeCamp:master Jun 13, 2016
@BerkeleyTrue BerkeleyTrue removed the QA label Jun 13, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants