-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Feature inline grid.js #11511
Feature inline grid.js #11511
Conversation
GCI discontinued
Eliminated some lines which were taking up more space and substituted them with a single line
Thanks for opening this pull request! This space is protected by our Code of Conduct - and we're here to help. |
Analysis results are not available for those commits View more on Code Climate. |
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.
Hi @Divine-peace, your PR contains commits from other contributors. Please amend this. Thank you!
Okay thanks
…On Wed, Oct 19, 2022, 12:23 PM Tilda Udufo ***@***.***> wrote:
***@***.**** requested changes on this pull request.
Hi @Divine-peace <https://github.com/Divine-peace>, your PR contains
commits from other contributors. Please amend this. Thank you!
—
Reply to this email directly, view it on GitHub
<#11511 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYUXTBVST53677QUSTM44DTWD7KZNANCNFSM6AAAAAARIGXMOE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hi @Divine-peace, could you close this PR and open a new one? Make sure you pull in the recent changes, create a new branch and then add your changes. Thank you! |
Okay
…On Fri, Oct 21, 2022, 12:22 PM Tilda Udufo ***@***.***> wrote:
Hi @Divine-peace <https://github.com/Divine-peace>, could you close this
PR and open a new one? Make sure you pull in the recent changes, create a
new branch and then add your changes. Thank you!
—
Reply to this email directly, view it on GitHub
<#11511 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYUXTBQIHNYNY7ELEIP4KZ3WEJ4HHANCNFSM6AAAAAARIGXMOE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hi @Divine-peace, do you need help with this? |
Yes
…On Tue, Oct 25, 2022, 7:30 AM Tilda Udufo ***@***.***> wrote:
Hi @Divine-peace <https://github.com/Divine-peace>, do you need help with
this?
—
Reply to this email directly, view it on GitHub
<#11511 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYUXTBSTITWZVSKZM2LYMNTWE55ADANCNFSM6AAAAAARIGXMOE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
You can edit the file directly from the web by using this link https://github.com/publiclab/plots2/edit/main/spec/javascripts/inline_grids_spec.js. You can follow this guide if you get stuck https://docs.github.com/en/repositories/working-with-files/managing-files/editing-files#editing-files-in-another-users-repository |
Thanks so much
…On Tue, Oct 25, 2022, 6:20 PM Tilda Udufo ***@***.***> wrote:
You can edit the file directly from the web by using this link
https://github.com/publiclab/plots2/edit/main/spec/javascripts/inline_grids_spec.js.
You can follow this guide if you get stuck
https://docs.github.com/en/repositories/working-with-files/managing-files/editing-files#editing-files-in-another-users-repository
—
Reply to this email directly, view it on GitHub
<#11511 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYUXTBRTPHM75NSNDFFMUILWFAJERANCNFSM6AAAAAARIGXMOE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Resolved in #11552 |
Changed from var to let
Fixes #11487
Make sure these boxes are checked before your pull request (PR) is ready to be reviewed and merged. Thanks!
rake test
@publiclab/reviewers
for help, in a comment below