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

feat(api-v1.1): enable publish API v1.1 #87

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

vernak2539
Copy link

@vernak2539 vernak2539 commented Dec 16, 2024

Start to enable v1.1 of the Publish API, fixing #86.

🚨 This should not be merged until after 31 December 2025.

I was 100% unsure about how to actually test this out / use this repo (pnpm workspace issues) when cloning this by itself. I'm not sure what key file to use in the test as well.

Happy to understand this a bit more and modify the code.

@louisgv if there's any guidance on this, it would be great so I can update docs/readme and contribute more effectively.

Copy link

@sankalpmukim sankalpmukim left a comment

Choose a reason for hiding this comment

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

We probably need to improve the documentation page where the users are guided on how to obtain the tokens. But codewise, LGTM! 🚀🚀

@vernak2539
Copy link
Author

We probably need to improve the documentation page where the users are guided on how to obtain the tokens. But codewise, LGTM! 🚀🚀

Good shout! I've just updated the README

@vernak2539
Copy link
Author

I've also submitted these two PRs to help enable it on the plasmo ecosystem

I think I know understand the bpp monorepo/gitmodule setup, but think I'll still need some guidance from @louisgv on getting it all merged come the new year.

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.

2 participants