-
Notifications
You must be signed in to change notification settings - Fork 130
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
updated privacy link fix #366 #413
Conversation
Thanks for opening this pull request! This space is protected by our Code of Conduct - and we're here to help. |
I do not get that. What I need to do next?? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes! This is great!! The code is properly updated. Just a couple notes for next time: First, put Issue #000
in the pr message itself, not the title. This is because github creates a link for issues mentioned in the posts, and that helps us keep organized! And second I notice that you used the branch name patch-2
, we ask that you use descriptive names. But this is just fine for this one! Thanks so much!
ohk! One more question that the way i did it was right?
I ll remeber to use descriptive ways in my prs.
Thank you so much
…On Mon, Mar 9, 2020 at 11:31 PM Natalie St Jean ***@***.***> wrote:
***@***.**** approved this pull request.
Yes! This is great!! The code is properly updated. Just a couple notes for
next time: First, put Issue #000 in the pr message itself, not the title.
This is because github creates a link for issues mentioned in the posts,
and that helps us keep organized! And second I notice that you used the
branch name patch-2, we ask that you use descriptive names.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#413?email_source=notifications&email_token=ALEPBC7ABITABLKRFBT3UVDRGU4IBA5CNFSM4LDYG24KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCYRT5WA#pullrequestreview-371408600>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALEPBC345D3Q3O26VEFCDX3RGU4IBANCNFSM4LDYG24A>
.
|
Hi @anshukaira. Congratulations on your first completed issue here on Public Lab. It depends on your skillset. You can try a few other beginner issues and once you fill confident enough, you can move onto the harder ones. If you have any question feel free to ask me or @nstjean. |
@anshukaira Yes it was right! Congrats! 🎉 |
This is for issue #366 |
travis taking forever...restarting |
Thanks @anshukaira for this 🚀 : and thanks @debck @nstjean and @keshav234156 for the reviews 🎉 |
Congrats on merging your first pull request! 🙌🎉⚡️ Help others take their first stepNow that you've merged your first pull request, you're the perfect person to help someone else out with this challenging first step. 🙌 Try looking at this list of `first-timers-only` issues, and see if someone else is waiting for feedback, or even stuck! 😕 People often get stuck at the same steps, so you might be able to help someone get unstuck, or help lead them to some documentation that'd help. Reach out and be encouraging and friendly! 😄 🎉 Read about how to help support another newcomer here, or find other ways to offer mutual support here. |
* FIX: adds missing entries in .gitignore adds .vscode in .gitignore adds _SpecRunner.html in .gitignore * updated privacy link (#413) Co-authored-by: Cess <cessmbuguar@gmail.com> Co-authored-by: shreyaa-s <s.shreyaa84429@gmail.com> Co-authored-by: Emily Ashley <15912063+emilyashley@users.noreply.github.com> Co-authored-by: Govind Goel <52847415+govindgoel@users.noreply.github.com> Co-authored-by: Kaira Hiwatari <46723211+anshukaira@users.noreply.github.com>
* updated privacy link (#413) Co-authored-by: Cess <cessmbuguar@gmail.com> * FIX: adds missing entries in .gitignore (#423) adds .vscode in .gitignore adds _SpecRunner.html in .gitignore Co-authored-by: Emily Ashley <15912063+emilyashley@users.noreply.github.com> Co-authored-by: Govind Goel <52847415+govindgoel@users.noreply.github.com> Co-authored-by: Cess <cessmbuguar@gmail.com> * added eslint, Husky and lint-staged Co-authored-by: Kaira Hiwatari <46723211+anshukaira@users.noreply.github.com> Co-authored-by: Cess <cessmbuguar@gmail.com> Co-authored-by: Shreya Sharma <55884943+shreyaa-s@users.noreply.github.com> Co-authored-by: Emily Ashley <15912063+emilyashley@users.noreply.github.com> Co-authored-by: Govind Goel <52847415+govindgoel@users.noreply.github.com>
Fixes #366
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
grunt jasmine
fixes #0000
-style reference to original issue #@publiclab/reviewers
for help, in a comment belowIf tests do fail, click on the red
X
to learn why by reading the logs.Please be sure you've reviewed our contribution guidelines at https://publiclab.org/contributing-to-public-lab-software
We have a loose schedule of reviewing and pulling in changes every Tuesday and Friday, and publishing changes on Fridays.
Thanks!