Skip to content

Feature/add feedback endpoints - #98

Merged
ivan-risueno merged 7 commits into
masterfrom
feature/add-feedback-endpoints
Oct 28, 2026
Merged

ivan-risueno merged 7 commits into
masterfrom
feature/add-feedback-endpoints

Conversation

@ivan-risueno

Copy link
Copy Markdown
Contributor

No description provided.

PROVISIONER_BITBUCKET_BRANCH_NAME=refs/heads/master

DEVSTACK_MARKETPLACE_API_PERMITTED_OIDS="any-valid-odis-for-token" No newline at end of file
DEVSTACK_MARKETPLACE_API_PERMITTED_OIDS="any-valid-odis-for-token"AZURE_TOKEN_URL="https://login.microsoftonline.com/test-tenant/oauth2/v2.0/token"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Missing return?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Seems that I misstyped🤔

var mockServer = MockRestServiceServer.bindTo(restTemplate).build();
mockServer.expect(requestTo("http://feedback.example.com/api/v1/feedback/submissions"))
.andExpect(method(POST))
.andRespond(withSuccess("{\"projectKey\":\"IVRI\"}", MediaType.APPLICATION_JSON));

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Although isn't problematic... better choose another name por project key (FOO,...)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

True, done.

@sonarqubecloud

Copy link
Copy Markdown

@ivan-risueno
ivan-risueno merged commit 3f5e000 into master Oct 28, 2026
3 checks passed
@ivan-risueno
ivan-risueno deleted the feature/add-feedback-endpoints branch October 28, 2026 10:51
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