-
Notifications
You must be signed in to change notification settings - Fork 63
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
Assets Service - UX to add/delete/modify files and folders on the Node-RED filesystem #4398
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 4357-file-api #4398 +/- ##
==============================================
Coverage 78.22% 78.22%
==============================================
Files 294 294
Lines 13848 13848
Branches 3117 3117
==============================================
Hits 10833 10833
Misses 3015 3015
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
…sages to be shown
…e its components if unavailable
I've put in place some checks that prevent unnecessary api calls to the assets endpoint when the feature is disabled or the nr-launcher version does not match the required version. sample.mov |
Also treated the use case where the instance is suspended/suspending/starting sample.mov |
Description
Todo
Screen.Recording.2024-08-23.at.17.01.38.mov
Requirements
add-file-api
file-api
Follow Up
Allow for users to control "visibility"/"share" functionality of each folder so that it becomes available via HTTP endpoints/Dashboard
Related Issue(s)
First part of #4358