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

link zum ical #398

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

link zum ical #398

wants to merge 4 commits into from

Conversation

tabascoeye
Copy link
Member

No description provided.

@silsha
Copy link
Member

silsha commented May 11, 2023

@tabascoeye Der Link funktioniert nur, wenn man eingeloggt ist. Glaube die .ics als Download anbieten ist auch recht nutzlos, weil das nur ein Export des aktuellen Stand ist (und sich somit nicht aktualisiert).

Der Link öffnet den "Kalender abonnieren"-Dialog in meiner App: webcal://p.fruux.com/c/a3298259302/81bae0fb-601c-47ee-ba92-460866466d90.ics

grafik

@tabascoeye
Copy link
Member Author

ah cool. Dann nehmen wir den? Oder geht der auch nur wenn eingeloggt?

@silsha
Copy link
Member

silsha commented May 12, 2023

Der geht ohne Login. Weiß nur nicht wie weit verbreitet webcal:// ist - zumindest Apple und Outlook scheinen es zu können.

tabascoeye and others added 2 commits June 2, 2023 16:31
use link which is public
* better deploy error handling (#400)

* better error-handling in deploy.sh

* Skip deploy if we are not in the base repo

Because PRs from other repos will not get access to the deploy SSH-Key anyway and this way we can output a proper error message

* Add GnoPN23 (#401)

Co-authored-by: Florian 'Cheatha' Köhler <[email protected]>

* move the error handling back to deploy.sh (#402)

its easyier to test and less confusing that way

---------

Co-authored-by: Ranlvor <[email protected]>
Co-authored-by: Florian 'Cheatha' Köhler <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants