-
Notifications
You must be signed in to change notification settings - Fork 56
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(devex): Local mocking of calls to the national registry #15523
Conversation
WalkthroughThe recent changes introduce a mock service for the National Registry endpoints, facilitating local development and testing. The updates include a new mock configuration file and a README guide on using these mocks. Additionally, the Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add Documentation and Community
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 3
Outside diff range, codebase verification and nitpick comments (3)
mocks/README.md (3)
5-5
: Consider adding a comma for clarity.The sentence might benefit from a comma after "port
8082
" for better readability.- Since the mock will be listening on port `8081` the port forwarding for xroad needs to be listening on port `8082` as that is where the mock server will forward requests it does not have mock responses for Since the mock will be listening on port `8081`, the port forwarding for xroad needs to be listening on port `8082` as that is where the mock server will forward requests it does not have mock responses forTools
LanguageTool
[uncategorized] ~5-~5: Possible missing comma found.
Context: ...t is where the mock server will forward requests it does not have mock responses for ##...(AI_HYDRA_LEO_MISSING_COMMA)
13-13
: Add a comma for grammatical correctness.There should be a comma after the introductory phrase "Currently".
- Currently only mock responses for the national registry are included. Currently, only mock responses for the national registry are included.
25-25
: Consider adding a comma for clarity.Adding a comma after "response time" could improve the readability of the sentence.
- No, one of the benefits of mocking locally is way shorter response time and to get that we need to mock even if the underlying service is up. No, one of the benefits of mocking locally is way shorter response time, and to get that we need to mock even if the underlying service is up.Tools
LanguageTool
[uncategorized] ~25-~25: Possible missing comma found.
Context: ...mocking locally is way shorter response time and to get that we need to mock even if...(AI_HYDRA_LEO_MISSING_COMMA)
[style] ~25-~25: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...horter response time and to get that we need to mock even if the underlying service is ...(REP_NEED_TO_VB)
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files ignored due to path filters (1)
yarn.lock
is excluded by!**/yarn.lock
,!**/*.lock
Files selected for processing (3)
- mocks/README.md (1 hunks)
- mocks/national-registry.json (1 hunks)
- package.json (2 hunks)
Additional context used
LanguageTool
mocks/README.md
[uncategorized] ~5-~5: Possible missing comma found.
Context: ...t is where the mock server will forward requests it does not have mock responses for ##...(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~12-~12: A comma may be missing after the conjunctive/linking adverb ‘Currently’.
Context: ...k-national-registry` ## Current mocks Currently only mock responses for the national re...(SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA)
[style] ~21-~21: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...Gervimaður Færeyjar fake person. If you need to mock other fake prersons you can downlo...(REP_NEED_TO_VB)
[uncategorized] ~21-~21: Possible missing comma found.
Context: ... person. If you need to mock other fake prersons you can download the [mockoon app](http...(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~25-~25: Possible missing comma found.
Context: ...mocking locally is way shorter response time and to get that we need to mock even if...(AI_HYDRA_LEO_MISSING_COMMA)
[style] ~25-~25: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...horter response time and to get that we need to mock even if the underlying service is ...(REP_NEED_TO_VB)
Additional comments not posted (3)
package.json (2)
340-340
: Addition of@mockoon/cli
.The addition of
@mockoon/cli
supports the new mocking functionality. Ensure that this tool integrates well with the existing development workflow and that its version is compatible with other project dependencies.
184-184
: Major version update forframer-motion
.The update from
6.5.1
to9.0.0
is a major version change. It's important to ensure that this doesn't introduce any breaking changes or compatibility issues with existing animations.mocks/national-registry.json (1)
1-705
: Review of the mock configuration file.The configuration appears comprehensive and well-structured. However, ensure that the use of wildcards in CORS settings (
Access-Control-Allow-Origin: *
) is intentional and acceptable for your security policy.
Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 6
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files ignored due to path filters (1)
yarn.lock
is excluded by!**/yarn.lock
,!**/*.lock
Files selected for processing (2)
- mocks/README.md (1 hunks)
- package.json (1 hunks)
Files skipped from review as they are similar to previous changes (1)
- package.json
Additional context used
LanguageTool
mocks/README.md
[uncategorized] ~5-~5: Possible missing comma found.
Context: ...t is where the mock server will forward requests it does not have mock responses for ##...(AI_HYDRA_LEO_MISSING_COMMA)
[uncategorized] ~12-~12: A comma may be missing after the conjunctive/linking adverb ‘Currently’.
Context: ...k-national-registry` ## Current mocks Currently only mock responses for the national re...(SENT_START_CONJUNCTIVE_LINKING_ADVERB_COMMA)
[style] ~21-~21: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...Gervimaður Færeyjar fake person. If you need to mock other fake persons, you can downlo...(REP_NEED_TO_VB)
[uncategorized] ~25-~25: Possible missing comma found.
Context: ...mocking locally is way shorter response time and to get that we need to mock even if...(AI_HYDRA_LEO_MISSING_COMMA)
[style] ~25-~25: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...horter response time and to get that we need to mock even if the underlying service is ...(REP_NEED_TO_VB)
Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A good start. I added some comments for you to review.
I'm wondering if there could be a simple path to ensuring that the mocks have the same schema as the clients to make future additions and maintenance simpler.
Thinking 'out loud' can we auto generate the mock files from the client json and perhaps use examples as mock data? If generating them automatically is unrealistic, can we do it programmatically so we can import the generated model types, in this case from libs/clients/national-registry/v2/gen/fetch/models
?
It looks like we already are using a tool for mocking APIs, Mountebank. Currently used to mock xroad in tests |
As of May 19th of this year mountebank is no longer being maintained. See the project's readme. Mockoon is also very user friendly, offering gui apps, a cli and a docker container. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 7
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (1)
- mocks/README.md (1 hunks)
Additional context used
LanguageTool
mocks/README.md
[uncategorized] ~13-~13: Possible missing comma found.
Context: ...ffic to port8082
. For more in depth instructions you can check out the [mockoon site](ht...(AI_HYDRA_LEO_MISSING_COMMA)
Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (1)
mocks/README.md (1)
7-7
: Add missing commas for clarity.There seems to be a missing comma after "Alternatively" which can improve the readability of the instructions.
- Alternatively if you use kubectl and socat just replace `8081:80` with `8082:80`. Alternatively, if you use kubectl and socat, just replace `8081:80` with `8082:80`.Tools
LanguageTool
[uncategorized] ~7-~7: A comma might be missing here.
Context: ... like soyarn proxies xroad --p 8082
. Alternatively if you use kubectl and socat just repla...(AI_EN_LECTOR_MISSING_PUNCTUATION_COMMA)
[uncategorized] ~7-~7: A comma might be missing here.
Context: .... Alternatively if you use kubectl and socat just replace
8081:80with
8082:80`. ...(AI_EN_LECTOR_MISSING_PUNCTUATION_COMMA)
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (1)
- mocks/README.md (1 hunks)
Additional context used
LanguageTool
mocks/README.md
[uncategorized] ~7-~7: A comma might be missing here.
Context: ... like soyarn proxies xroad --p 8082
. Alternatively if you use kubectl and socat just repla...(AI_EN_LECTOR_MISSING_PUNCTUATION_COMMA)
[uncategorized] ~7-~7: A comma might be missing here.
Context: .... Alternatively if you use kubectl and socat just replace
8081:80with
8082:80`. ...(AI_EN_LECTOR_MISSING_PUNCTUATION_COMMA)
Additional comments not posted (6)
mocks/README.md (6)
5-7
: Clarify port forwarding setup.The instructions on setting up port forwarding are clear, but it might be beneficial to include a note about why port
8082
is specifically needed, especially since8081
is used for Mockoon. This would help developers understand the network setup better.Would you like me to add this clarification?
Tools
LanguageTool
[uncategorized] ~7-~7: A comma might be missing here.
Context: ... like soyarn proxies xroad --p 8082
. Alternatively if you use kubectl and socat just repla...(AI_EN_LECTOR_MISSING_PUNCTUATION_COMMA)
[uncategorized] ~7-~7: A comma might be missing here.
Context: .... Alternatively if you use kubectl and socat just replace
8081:80with
8082:80`. ...(AI_EN_LECTOR_MISSING_PUNCTUATION_COMMA)
11-11
: Clarify the mock server startup command.It might be helpful to specify where this command should be run or if there are any specific directory or setup prerequisites.
Would you like me to add more detailed instructions on where and how to run this command?
17-17
: Specify version for clarity.It's mentioned that only mock responses for the national registry V2 are included. It might be useful to specify what aspects of V2 are covered to avoid any confusion for developers who are unfamiliar with the version details.
Would you like help detailing the V2 coverage in the mocks?
21-21
: Expand on transparent proxying.The description of how Mockoon handles unmocked requests is succinct. However, adding a brief explanation of how transparent proxying works could provide valuable context for developers new to the concept.
Would you like me to include a short explanation or diagram here?
25-25
: Clarify the setup for additional mocks.The instructions for setting up additional mocks are clear, but it could be beneficial to include examples of common scenarios or links to tutorials for less experienced users.
Would you like me to add examples or further resources?
29-29
: Highlight the benefits of local mocking.The last paragraph does a good job of explaining the benefits of local mocking. Consider adding a brief note on how this can also help in scenarios where the national registry is unavailable due to maintenance or outages, enhancing the understanding of its utility.
Would you like me to draft this addition?
Datadog ReportAll test runs ✅ 101 Total Test Services: 0 Failed, 99 Passed Test ServicesThis report shows up to 10 services
🔻 Code Coverage Decreases vs Default Branch (2) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the mock not belong where the client code is located?
And as I said in a previous comment we should try to use the types we have as much as possible. Take a look at this reference for example.
Here there is a function imported from the gen
folder to generate the JSON to return.
import { EinstaklingsupplysingarToJSON } from '@island.is/clients/national-registry-v2'
This way if there is ever a change in the clientConfig.json
the mock file would not go out of sync and the response will always be correct.
Well the thinking was that this might function as a sort of central repo for not just the national registry v2 client but other clients as well as that might aid discoverability. However should the core team feel that the capture files belong with the clients themselves that is not a problem.
The thing is that the json file included here is a generated file so types don't really apply. Should the types change, the capture need simply be re-generated. I did however change the readme to hopefully make the fact that the json files are generated more clear. |
* Add mockoon cli and national registry mock json * Add readme * Fix yarn.lock conflict * Roll back to main version of package.json and yarn.lock * Re-add mockoon cli after revert to main version * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Re-adding lost yarn script * Remove mockoon cli * yarn lock * Re-add script * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Fix rabbit fix * Rolled package.json and yarn.lock back to main and addressed pr comments * Another revert attempt * chore: nx format:write update dirty files * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Add clarity to radme and rename capture file to clarify that it is for version 2 only. * More clarification in readme * Add mockoon CLI as dev dependency * Move and rename data file for added clarity * chore: nx format:write update dirty files --------- Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> Co-authored-by: andes-it <[email protected]> Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com>
* Add mockoon cli and national registry mock json * Add readme * Fix yarn.lock conflict * Roll back to main version of package.json and yarn.lock * Re-add mockoon cli after revert to main version * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Re-adding lost yarn script * Remove mockoon cli * yarn lock * Re-add script * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Fix rabbit fix * Rolled package.json and yarn.lock back to main and addressed pr comments * Another revert attempt * chore: nx format:write update dirty files * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Add clarity to radme and rename capture file to clarify that it is for version 2 only. * More clarification in readme * Add mockoon CLI as dev dependency * Move and rename data file for added clarity * chore: nx format:write update dirty files --------- Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> Co-authored-by: andes-it <[email protected]> Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com>
…15794) * Adding bearer auth to appropriate controllers * Reverting changes in non application-system code * fix(application-ir): Executor Quantifier (#15790) * fix(service-portal): Finance mobile padding (#15640) Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * chore(occupational-licenses): Display license number in health directorate licenses (#15775) * chore: init * chore: fix version * chore: remove console.log * chore: revert some --------- Co-authored-by: Þorkell Máni Þorkelsson <[email protected]> Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * feat(contentful-entry-tagger): Use team name as entry tagging substitute (#15791) Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * feat(devex): Local mocking of calls to the national registry (#15523) * Add mockoon cli and national registry mock json * Add readme * Fix yarn.lock conflict * Roll back to main version of package.json and yarn.lock * Re-add mockoon cli after revert to main version * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Re-adding lost yarn script * Remove mockoon cli * yarn lock * Re-add script * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Fix rabbit fix * Rolled package.json and yarn.lock back to main and addressed pr comments * Another revert attempt * chore: nx format:write update dirty files * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Add clarity to radme and rename capture file to clarify that it is for version 2 only. * More clarification in readme * Add mockoon CLI as dev dependency * Move and rename data file for added clarity * chore: nx format:write update dirty files --------- Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> Co-authored-by: andes-it <[email protected]> Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * fix: url printing in ci (#15788) * force deploy * updated yargs * reverted safe vars * debugging comment message * testing again * removed debug printing * removed debug printing * checking file contents * testing docker run script in CI * removed logger debug and changed logger * removed deploy --------- Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * feat(web): Profession rights list - Display national id field in table view (#15792) * Add national id field * Cast as any * Ignore type warnings due to pipeline not being up to date with types --------- Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * Adding bearer auth to payment status endpoint * Removing unneeded auth param from request, now done automatically * chore: nx format:write update dirty files --------- Co-authored-by: kksteini <77672665 [email protected]> Co-authored-by: Ásdís Erna Guðmundsdóttir <[email protected]> Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> Co-authored-by: Þorkell Máni Þorkelsson <[email protected]> Co-authored-by: Þorkell Máni Þorkelsson <[email protected]> Co-authored-by: Rúnar Vestmann <43557895 [email protected]> Co-authored-by: norda-gunni <161026627 [email protected]> Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> Co-authored-by: andes-it <[email protected]> Co-authored-by: brynjarorng <[email protected]>
* Add mockoon cli and national registry mock json * Add readme * Fix yarn.lock conflict * Roll back to main version of package.json and yarn.lock * Re-add mockoon cli after revert to main version * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Re-adding lost yarn script * Remove mockoon cli * yarn lock * Re-add script * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Fix rabbit fix * Rolled package.json and yarn.lock back to main and addressed pr comments * Another revert attempt * chore: nx format:write update dirty files * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Add clarity to radme and rename capture file to clarify that it is for version 2 only. * More clarification in readme * Add mockoon CLI as dev dependency * Move and rename data file for added clarity * chore: nx format:write update dirty files --------- Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> Co-authored-by: andes-it <[email protected]> Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com>
…15794) * Adding bearer auth to appropriate controllers * Reverting changes in non application-system code * fix(application-ir): Executor Quantifier (#15790) * fix(service-portal): Finance mobile padding (#15640) Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * chore(occupational-licenses): Display license number in health directorate licenses (#15775) * chore: init * chore: fix version * chore: remove console.log * chore: revert some --------- Co-authored-by: Þorkell Máni Þorkelsson <[email protected]> Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * feat(contentful-entry-tagger): Use team name as entry tagging substitute (#15791) Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * feat(devex): Local mocking of calls to the national registry (#15523) * Add mockoon cli and national registry mock json * Add readme * Fix yarn.lock conflict * Roll back to main version of package.json and yarn.lock * Re-add mockoon cli after revert to main version * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Re-adding lost yarn script * Remove mockoon cli * yarn lock * Re-add script * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Fix rabbit fix * Rolled package.json and yarn.lock back to main and addressed pr comments * Another revert attempt * chore: nx format:write update dirty files * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Update mocks/README.md Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> * Add clarity to radme and rename capture file to clarify that it is for version 2 only. * More clarification in readme * Add mockoon CLI as dev dependency * Move and rename data file for added clarity * chore: nx format:write update dirty files --------- Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> Co-authored-by: andes-it <[email protected]> Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * fix: url printing in ci (#15788) * force deploy * updated yargs * reverted safe vars * debugging comment message * testing again * removed debug printing * removed debug printing * checking file contents * testing docker run script in CI * removed logger debug and changed logger * removed deploy --------- Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * feat(web): Profession rights list - Display national id field in table view (#15792) * Add national id field * Cast as any * Ignore type warnings due to pipeline not being up to date with types --------- Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> * Adding bearer auth to payment status endpoint * Removing unneeded auth param from request, now done automatically * chore: nx format:write update dirty files --------- Co-authored-by: kksteini <77672665 [email protected]> Co-authored-by: Ásdís Erna Guðmundsdóttir <[email protected]> Co-authored-by: kodiakhq[bot] <49736102 kodiakhq[bot]@users.noreply.github.com> Co-authored-by: Þorkell Máni Þorkelsson <[email protected]> Co-authored-by: Þorkell Máni Þorkelsson <[email protected]> Co-authored-by: Rúnar Vestmann <43557895 [email protected]> Co-authored-by: norda-gunni <161026627 [email protected]> Co-authored-by: coderabbitai[bot] <136622811 coderabbitai[bot]@users.noreply.github.com> Co-authored-by: andes-it <[email protected]> Co-authored-by: brynjarorng <[email protected]>
...
Attach a link to issue if relevant
What
Added a local mock proxy to enable local mocking of requests to the national registry.
Why
The national registry api tends to be kind of slow to respond to requests so enabling local mocking makes for a much snappier local dev experience. A side benefit is less load on the national registry dev env.
The same setup can also be used to mock any other service.
Screenshots / Gifs
Attach Screenshots / Gifs to help reviewers understand the scope of the pull request
Checklist:
Summary by CodeRabbit
New Features
Dependencies
framer-motion
from6.5.1
to9.0.0
.@mockoon/cli
dependency (version8.3.0
).