Skip to content
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

calendar should show event location 😅 #157

Closed
jni opened this issue Feb 28, 2024 · 6 comments · Fixed by napari/docs#455
Closed

calendar should show event location 😅 #157

jni opened this issue Feb 28, 2024 · 6 comments · Fixed by napari/docs#455
Labels
enhancement New feature or request
Milestone

Comments

@jni
Copy link
Member

jni commented Feb 28, 2024

This event has imagesc island attached to it as a location, but that turned out to be futile... 😭

Screenshot 2024-02-28 at 9 44 41 pm
@Czaki
Copy link
Contributor

Czaki commented Feb 29, 2024

It is strictly mobile problem:
obraz

@melissawm
Copy link
Member

melissawm commented Feb 29, 2024

Hmm I'll need to check this. This is what I get on Firefox mobile:

And Chrome mobile:

Both on Android. Maybe there is some setting to turn on scrolling on smaller screens that I need to activate.

@melissawm
Copy link
Member

Oh wait, the text is different in @jni's screenshot! Maybe it's a calendar caching issue and it has by now been updated in Google Calendar?

@psobolewskiPhD
Copy link
Member

Looks fine in mobile safari:
IMG_3F319E0B1DF0-1

But I assume Juan just added it to the details of the event. There is no location field in any of the events mobile or not.

@melissawm melissawm added the enhancement New feature or request label Jun 28, 2024
@melissawm melissawm added this to the v0.4.1 milestone Jun 28, 2024
@melissawm
Copy link
Member

I am looking into this and it seems like the best strategy would be to have some sort of structure for our meeting invites and use that for parsing. Otherwise I'm afraid this is going to get more complicated and hard to maintain. So if someone with access to the calendar (maybe @jni ) could go and make all of them look like e.g.:

Community meeting

Description ...

Agenda: <link>
Zoom link: <link>
Location: <link>

then I could make those into nicely formatted fields in the calendar.

@jni
Copy link
Member Author

jni commented Jul 11, 2024

@melissawm I don't understand why arbitrary text fields are better than just grabbing the actual calendar event field for this? (Other than the fact that existing code already extracts the description.) The only thing is whether we do want to make sure that the description has all the info, in which case the other fields are redundant.

Anyway, I can update the descriptions. I'll just have "Agenda" and "Call link" so that we can switch providers at any time. 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Development

Successfully merging a pull request may close this issue.

4 participants