Skip to content
This repository has been archived by the owner on Jan 10, 2024. It is now read-only.

[Feature] Calendar notification does nothing when clicked #1441

Closed
kmodexc opened this issue May 10, 2022 · 2 comments · Fixed by #1444
Closed

[Feature] Calendar notification does nothing when clicked #1441

kmodexc opened this issue May 10, 2022 · 2 comments · Fixed by #1444

Comments

@kmodexc
Copy link
Contributor

kmodexc commented May 10, 2022

It would be nice, if it would open the location of the lecture. I would like it if it opens nav.tum.sexy with the location.

@kmodexc kmodexc changed the title Calendar notification does nothing when clicked [Feature] Calendar notification does nothing when clicked May 10, 2022
@CommanderStorm
Copy link
Member

Opening in-app, if #1437 is done would be good, too :)
Going off-platform is not nessesesarily the best UX, but this is just Personal preference.
Going by the ammount of cool features that we are developing currently redirecting Off-Platform could be a good solution, until the campusapss have catched up.

However the Locations are pretty cryptic sometimes.
Does anyone have a longer samplesize of what possible values are? (for a test, how reliable we are and where we have to implement some extra logic)

@kmodexc
Copy link
Contributor Author

kmodexc commented May 16, 2022

Opening in-app, if #1437 is done would be good, too :) Going off-platform is not nessesesarily the best UX, but this is just Personal preference. Going by the ammount of cool features that we are developing currently redirecting Off-Platform could be a good solution, until the campusapss have catched up.

However the Locations are pretty cryptic sometimes. Does anyone have a longer samplesize of what possible values are? (for a test, how reliable we are and where we have to implement some extra logic)

I did that in the PR mentioned.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants