Closed daemon1024 closed 3 years ago
Thanks for opening this pull request! Dangerbot
will test out your code and reply in a bit with some pointers and requests.
There may be some errors, but don't worry! We're here to help! 👍🎉😄
It works 😄
/cc @jywarren
//offtopic Btw just noticed, we could use some updates on welcome bot messages. Potential good first issues, across repositories.
//offtopic Btw just noticed, we could use some updates on welcome bot messages. Potential good first issues, across repositories.
great idea.
OK, and this one is good to merge?
Looks like it! Thanks!!!
Would you like to add a package.json version increment of 0.0.1
?
and run npm install
so we get an updated package-lock.json too? Then I'll be able to publish it as a new version on NPM
Done :)
Congrats on merging your first pull request! 🙌🎉⚡️ Your code will likely be published to https://publiclab.org in the next few days. In the meantime, can you tell us your Twitter handle so we can thank you properly? Now that you've completed this, you can help someone else take their first step! See: Public Lab's coding community!
As soon as this publishes, let's link and follow to the PR in LEL, and then same to plots2 -- sound good? Thank you!!! Also doesn't have to be tonight, it's so late!
Yup, Sounds good to me. Thank You and Good Night 🌃
Published! Good night!!!
If dependabot doesn't open it, we can open it manually!
OK! We more or less got it... still seeing an "undefined" but very close!!! Did we use the wrong property name?
Hmm.
{
"doc_id": 768638,
"doc_type": "PLACES",
"doc_url": "/profile/charliejohnson",
"doc_title": "charliejohnson",
"doc_score": null,
"doc_author": null,
"doc_image_url": "https://www.gravatar.com/avatar/6e62cf8e8246c20b6b9839c175dc996a",
"latitude": "0",
"longitude": "0",
"blurred": true,
"category": "PLACES",
"place_name": null,
"created_at": "2021-08-10T11:07:07.000Z",
"time_since": null,
"comment_count": null
}
The url is right, just the title is undefined somehow 🤔
For the screenshot above, I was def. getting doc_title
values...
{
"doc_id": 766513,
"doc_type": "PLACES",
"doc_url": "/profile/dennlawgroup",
"doc_title": "dennlawgroup",
"doc_score": null,
"doc_author": null,
"doc_image_url": "/public/system/users/photos/000/766/513/thumb/denn-law-logo-1x-2.png",
"latitude": "42",
"longitude": "-71",
"blurred": true,
"category": "PLACES",
"place_name": null,
"created_at": "2021-07-21T08:33:27.000Z",
"time_since": null,
"comment_count": null
}
We override the JSONparser function in LEL!
Ohhh.
So just one more release then :D Making a PR asap.
Fixes #133