-
Notifications
You must be signed in to change notification settings - Fork 5.4k
[Components] Support file URLS and file paths in a single prop input #17169
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
base: master
Are you sure you want to change the base?
[Components] Support file URLS and file paths in a single prop input #17169
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 3 Skipped Deployments
|
Important Review skippedMore than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review. 67 files out of 174 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits. You can disable this status message by setting the ✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. 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 using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
3fa1972
to
cd649c0
Compare
@sergio-eliot-rodriguez @vunguyenhung @dannyroosevelt Once this PR gets merged I will need the following update in the platform:
Thanks in advance! |
Hello everyone, I have tested this PR and there're some test cases failed or needed improvement. Please check the test report below for more information |
a7720c7
to
d808a35
Compare
Hi @vunguyenhung here are my comments to your last test report! cloudmersive - Convert To Pdf - FailedYou need to compile the Typscript first then publish! $ npm run build
$ pd publish ./components/cloudmersive/dist/actions/convert-to-pdf/convert-to-pdf.mjs
|
d808a35
to
bf23731
Compare
Hello everyone, I have tested this PR and there're some test cases failed or needed improvement. Please check the test report below for more information |
bf23731
to
b4d363a
Compare
deepimage - Auto Enhance - FailedCan you make sure you are deploying the right version please. I couldn't reproduce any of your reported issues here! Maybe you can increase the version manually and deploy again! |
b4d363a
to
aaed479
Compare
components/discord/actions/send-message-with-file/send-message-with-file.mjs
Outdated
Show resolved
Hide resolved
components/discord/actions/send-message-with-file/send-message-with-file.mjs
Outdated
Show resolved
Hide resolved
aaed479
to
81b9005
Compare
@jcortes, just to clarify on the DeepImage situation:
My suggestion:
Please let me know any concern you may have. Thanks Best, |
Hi everyone, all test cases are passed! Ready for release! |
Yeah sorry for the confusion here! |
81b9005
to
660562c
Compare
WHY
Resolves #16977