-
Notifications
You must be signed in to change notification settings - Fork 0
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 support for lack of event's location #48
Labels
Comments
fabiodrg
added
enhancement
New feature or request
good first issue
Good for newcomers
labels
Feb 2, 2019
I don't think it has that limitation. In any case, one could always change it to I believe I left it as |
Ok, that makes some sense. Maybe this could be a setting in the options page? |
Agreed, and that should be straightforward to implement! |
fabiodrg
added a commit
that referenced
this issue
Mar 29, 2022
fabiodrg
added a commit
that referenced
this issue
Mar 29, 2022
fabiodrg
added a commit
that referenced
this issue
Mar 29, 2022
fabiodrg
added a commit
that referenced
this issue
Mar 29, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Not sure, but I think all extractors in Sigarra have some sort of location, typically a classroom. However, that assumption isn't verified in Moodle, and as a workaround the current location is Moodle. Would be nice to handle this more properly.
The Google Calendars link and Outlook don't require the location parameter. The standard for ICS almost sure it doesn't require it either, although not sure if the library we are using has limitations.
The text was updated successfully, but these errors were encountered: