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

feat: add external-mocks-application with skilllab mock #46

Closed
wants to merge 0 commits into from

Conversation

tomwwinter
Copy link
Contributor

No description provided.

@tomwwinter tomwwinter force-pushed the tw/feat/external-mock-service-application branch from a0480bd to 25a1835 Compare November 8, 2024 11:31
Copy link
Member

@sleidig sleidig left a comment

Choose a reason for hiding this comment

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

Do I understand correctly that this simulates the behavior of the third-party API and can be used for developing/testing without sending requests to the other platform?

Great stuff, looks pretty clear also.

}

@RestController
class HelloFriendController {
Copy link
Member

Choose a reason for hiding this comment

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

I am not sure where we can document this best but let's add some general code comment or readme briefly explaining what the purpose of aam-external-mock-service is. (possibly in a completly different file or folder also)

@tomwwinter tomwwinter closed this Dec 10, 2024
@tomwwinter tomwwinter force-pushed the tw/feat/external-mock-service-application branch from 208ad38 to de169dc Compare December 10, 2024 20:09
@tomwwinter tomwwinter deleted the tw/feat/external-mock-service-application branch December 10, 2024 20:09
@tomwwinter tomwwinter restored the tw/feat/external-mock-service-application branch December 10, 2024 20:09
@tomwwinter tomwwinter deleted the tw/feat/external-mock-service-application branch December 10, 2024 20:10
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