-
Notifications
You must be signed in to change notification settings - Fork 8
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
'bb announcements' displays all announcements when specified course is not found #28
Comments
Possible solutions:
|
Personally, I think it would be best to display an error message and exit without printing any announcements. |
On the bright side, this means that implementing the 'get announcements for all courses' feature will be extremely easy. Note that there's really 2 problems here:
For problem 1, I'd say it's probably best to display an error message. For option 1: git doesn't pipe to a pager only if there's a lot of output. Commands like Another possible option: limit to announcements since a user specified date, or within a user-specified number of days back. This will probably be more useful than option 2 but more difficult to implement. |
Good point, I didn't realize that. I agree then that piping through the pager always would be a good idea. (Probably also for For |
+1 pager. As another extention to option 2, I'd like the ability to see only new (unread) announcements, but this might be extremely hard to implement. |
The unread announcements is very similar to my idea of limiting by date - if bb stored the last date it got announcements for a course, then it could just limit by that date by default. |
I'm not sure if this is a bug or a feature, but having my terminal spammed with every BlackBoard announcement I've ever recieved is not optimal, IMHO.
The text was updated successfully, but these errors were encountered: