Closed mit-27 closed 1 week ago
Qovery can create a Preview Environment for this PR. To trigger its creation, please post a comment with one of the following command.
Command | Blueprint environment |
---|---|
/qovery []() preview 783d0240-ec38-4387-a9a9-8e225f1bd3e1 | dev |
/qovery []() preview {all|UUID1,UUID2,...} | To preview multiple environments |
This comment has been generated from Qovery AI :robot:. Below, a word from its wisdom :
Make it work, make it right, make it fast
Latest commit: bdf24c33904025496ab21b345e102a046d60235f
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
[!IMPORTANT]
Review skipped
Auto reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
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?
Demo
Tasks
- [x] Error handing for embedded catalog component
- [x] Api Key auth option for magic link and embedded catalog component
Awesome thanks ! Before I merge can you update the code when you display the form to input the API Key by adding other properties if it exists for the considered provider ?
You can retrieve the properties in the metadat.ts file as well:
You may need to add just a if condition if properties are needed and create other input for these fields as well as some APIs ask for more than just an api key for auth.
Demo
Tasks