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

Added question service and linked it with the gateway service #39

Merged
merged 1 commit into from
Mar 6, 2024

Conversation

ChristianFN2
Copy link
Contributor

Question service should be prepared for use.

Please, check if the linking in the gateway service is correct.

Also, if you are in charge of using the question service, check the documentation of the endpoints to know the format the responses are sent with and the required format when sending the requests (We can discuss if you find another format more appropriate/comfortable).

Copy link

sonarqubecloud bot commented Mar 6, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
38.5% Coverage on New Code (required ≥ 80%)
90.5% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@ChristianFN2
Copy link
Contributor Author

I should also add I installed version 8 of wikibase-sdk because it is the last version supporting 'require', but it should not cause any problems still and functionally it is practically the same. This is the documentation page for this version: https://github.com/maxlath/wikibase-sdk/tree/v8.1.1#as-a-module

@ChristianFN2 ChristianFN2 requested a review from andrrsin March 6, 2024 17:11
@andrrsin
Copy link
Contributor

andrrsin commented Mar 6, 2024

Seems as a pretty good microservice Christian. The library shouldn't be a problem, we'll write it down in the documentation! The rest looks just fine. And for the moment let's not worry about the tests as they were not programmed for our new API ;)

@andrrsin andrrsin merged commit 727c6d8 into develop Mar 6, 2024
1 of 2 checks passed
@andrrsin
Copy link
Contributor

andrrsin commented Mar 6, 2024

Quieres que borre la rama? @ChristianFN2

Copy link
Contributor

@andrrsin andrrsin left a comment

Choose a reason for hiding this comment

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

Everything seems perfect, great Job :D

@ChristianFN2 ChristianFN2 deleted the countryFlagQuestions branch March 7, 2024 06: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