Can You Review or Fast-Track My PR for Token Metrics API Support on Pipedream Integration?

This topic was automatically generated from Slack. You can find the original thread here.

Hi / @U02A06JPKEH
I’ve submitted a PR to add Token Metrics API support for the Pipedream integration. It has been pending approval for a few days now. Could you please help review or fast-track this so we can move forward with publishing it to the Pipedream marketplace? Thanks a lot! :pray:

Hey Thanks for the note. I moved your PR to status “Ready for PR Review”. There is a team checking PR in this status, they will get to this soon.

Only yours is in the queue, so that shouldn’t take long.

Checking the status on this — the PR is still in the approval process. Can we escalate this please? Also, I’d like to know how I can manage and change the logo of my integration. Currently, I don’t see any option for that. Could you please share some reference on how to do it?

Hi there , I see the reviewer left comments and the PR status has been moved to “Changes Required”. Would you mind tackling those?

Kindly share the logo you’d like to see for your integration. and I can have it change it

Here’s the logo you can use. Meanwhile, I’ll finish up the required changes

Hi we updated Token Metrics logo. Make sure to hard refresh your pages for the new logo to appear. Thanks for the collaboration!

Thanks I can see it now. I’ve implemented all the requested changes in the PR—could you please ask your team to revisit? It would be great if we can get this merged today.

Also, just to clarify since I’ve made the requested changes, do I need to republish the actions from my local, or will the updated code be deployed and reflected in the actions once it reaches your marketplace?

Hi , you only need to republish your components to test the updated code yourself. The marketplace components will be deployed with the latest code when the PR is merged.

Great, thanks for the update . Is the PR ready to merge now?

We’ll review your adjustments and let you know if any other changes are needed. The components will also be QA’d before they are ready to be merged

Great thanks !

@U03EGTE5WC8 any update on the release status of the PR? Can we check on this please?

Hey , I’ve updated the component version that was still unmodified. Can you please run this command on your branch, check/fix any errors and commit the results?

npx eslint --fix components/token_metrics

This is the last check failing, after the ESLint check passes we can proceed to merge the PR.

it’s done!

The PR has just been merged - thanks again for your contribution!