Skip to content

Conversation

@nishant-shah-social
Copy link

@nishant-shah-social nishant-shah-social commented Aug 23, 2024

Summary

While trying to setup flagsmith integration on the jira cloud instance, we faced an issue where-in an error was received while adding a flagsmith project in the jira project settings. Further debugging, we found that it gives an error Unsupported media type 422 while configuring flagsmith in the jira project settings.
Looking at the documentation of jira v3 api, we found that the concerned PUT api call requires the content-type and Accept request headers which were missing. After adding these headers we were able to configure flagsmith for jira cloud successfully.
The PR is to address this problem.

@nishant-shah-social nishant-shah-social changed the title fix: error in adding flagsmith property to jira project fix: error while adding flagsmith property to jira project Aug 23, 2024
@matthewelwell
Copy link
Contributor

Thanks for this @nishant-shah-social . Could you fix up the formatting error here and we can get this merged and released.

@nishant-shah-social
Copy link
Author

Thanks for this @nishant-shah-social . Could you fix up the formatting error here and we can get this merged and released.

Done @matthewelwell

@matthewelwell
Copy link
Contributor

Sorry @nishant-shah-social , not quite. You can see the necessary changes in the job failure here.

@matthewelwell matthewelwell changed the base branch from main to chore/fix-project-property-flagsmith September 27, 2024 13:40
@matthewelwell
Copy link
Contributor

@nishant-shah-social I'm just going to merge this into a local branch and fix the linting myself.

@matthewelwell matthewelwell merged commit 4613202 into Flagsmith:chore/fix-project-property-flagsmith Sep 27, 2024
1 check failed
@nishant-shah-social
Copy link
Author

@nishant-shah-social I'm just going to merge this into a local branch and fix the linting myself.

Thanks @matthewelwell! My bad! Looks like that was the only permutation I didn't try 😄

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