#120 Remove the port

[metinbaris]

Fixes #120

[metinbaris]

@pfwd Have one question, is it ok use it without url slash ?

There is "/" http://localhost/ at the end of url as described "#120 Acceptance Criteria"

To use the given port I had to change also test description for yarn test would please review it for me :)

[pfwd]

@pfwd Have one question, is it ok use it without url slash ?

There is "/" http://localhost/ at the end of url as described "#120 Acceptance Criteria"

To use the given port I had to change also test description for yarn test would please review it for me :)

Good question. I haven't tried it without the slash.
Now you mention it, I actually prefer the API_BASE_PATH to not have the trailing slash so we can define that in the routes.
It should be ok as

code-quiz/api-client/src/api/requestHandlers/quizRequest.ts

Line 4 in 9564ab3

return axiosClient.get('/quiz')

starts with the slash. This is covered by the following integration test

code-quiz/api-client/tests/integration/api/requestHandlers/quizRequest.test.ts

Line 14 in 9564ab3

const response = await getAll()

Could you try and remove it from the update the unit test? If the integration test fails then I may need to rethink how it works.

[metinbaris]

@pfwd Just pushed the change, yes if we remove the slash at the end of the baseURL, requests should work fine