-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add delete-s command to UG #49
Add delete-s command to UG #49
Conversation
✅ Deploy Preview for tp17 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Codecov Report
@@ Coverage Diff @@
## master #49 +/- ##
=========================================
Coverage 75.26% 75.26%
Complexity 419 419
=========================================
Files 71 71
Lines 1338 1338
Branches 126 126
=========================================
Hits 1007 1007
Misses 301 301
Partials 30 30 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
**Acceptable values for each parameter**: | ||
* `TUTOR_INDEX`: Only accepts numerical value | ||
**Acceptable values for each parameter:** | ||
* `SCHEDULE_INDEX`: NUMBER Only numerical input that ranges from 1 to the last schedule shown in the list of schedules. |
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.
This line should omit the NUMBER
SCHEDULE_INDEX
: Only numerical input that ranges from 1 to the last schedule shown in the list of schedules.
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.
LGTM for now. Lets do the remaining change in the future.
Add UI mock for delete-s command
Add information for delete-s command