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

comments are modified in this file #914

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

Conversation

yashsehgal
Copy link

I have modified the comment line in a javascript file, the comments were following HTML Syntax.
File Modified Path: src\assets\scripts\fbpixel.js

Waiting for your response.
Issue #913

@boss-contributions-bot
Copy link

Thanks @yashsehgal, for opening the pull request! 🙌

One of our mentors will review the pull request soon. ✅

Star ⭐ this project and tweet 🐦 about your contributions.

@codingblocks-bot
Copy link
Contributor

codingblocks-bot commented Oct 1, 2020

Deploy preview for cb-netlify-prod ready!

Built with commit 15dbe99

https://deploy-preview-914--cb-netlify-prod.netlify.app

…ccording to issue (coding-blocks#916: Typo mistake) and (coding-blocks#905: What out students say, is now displayed correctly in center)
Copy link
Author

@yashsehgal yashsehgal left a comment

Choose a reason for hiding this comment

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

Some punctuation mistakes are resolved in this file. Waiting for the review.

Copy link

@arnabuchiha arnabuchiha left a comment

Choose a reason for hiding this comment

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

This issue is fixed

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.

None yet

4 participants