Skip to content

Conversation

Zaimwa9
Copy link
Contributor

@Zaimwa9 Zaimwa9 commented Sep 15, 2025

We reverted the sdk-version header changes due to CORS errors in #342.

This PR re-integrates the headers while releasing a major version.
Additionally added a pre-commit hook to write version without having to build.

Contributes to #333

@Zaimwa9 Zaimwa9 requested a review from khvn26 September 15, 2025 08:51
@khvn26
Copy link
Member

khvn26 commented Sep 15, 2025

Additionally added a pre-commit hook to write version without having to build.

Nice! That will likely be retired once we close #321, though.

Copy link
Member

@khvn26 khvn26 left a comment

Choose a reason for hiding this comment

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

Please don't release before we close https://github.com/Flagsmith/github-webhook-handler/issues/10, or before I confirm the Github webhook handler is otherwise running.

@Zaimwa9
Copy link
Contributor Author

Zaimwa9 commented Sep 15, 2025

Yes @khvn26 , just wanted to have it ready 👍

@matthewelwell matthewelwell changed the title Feat/re-add sdk version headers feat!: re-add sdk version headers Sep 15, 2025
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