timlrx / pliny

Create content rich websites with ease - built on next.js, contentlayer and tailwindcss
MIT License
293 stars 42 forks source link

Updated URL of `beehiiv` API endpoint #179

Closed SahilFruitwala closed 1 month ago

SahilFruitwala commented 1 month ago

I think URL that is being used in beehiiv newsletter is wrong. Current documentation from beehiiv: docs

changeset-bot[bot] commented 1 month ago

⚠️ No Changeset found

Latest commit: 0fe6eb3b5f6fb398168e5d784126e6fd9f82b16f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

timlrx commented 1 month ago

Looks right to me, but tagging @lappemic who contributed to the code - did you manage to get it working previously? Or did something in the API change?

lappemic commented 1 month ago

it worked for me before.

Havent looked into it properly now. Can do the next week if the issue still exists.

AntonioDrk commented 1 month ago

@timlrx I had issues with beehiiv as well, this change helped me with the integration. Can we expect to see this in the next version of pinly?

lappemic commented 1 month ago

Just looked into it. I did the API url directly in the blog and it seems i did mess it up in pliny indeed! The fix suggested from @SahilFruitwala is correct!

Sorry for the inconveniences!

timlrx commented 1 month ago

Thanks for verifying! Let's merge this in and I will publish the fix soon. Released as v0.3.2