Open
Conversation
schliz
requested review from
hd1ex,
mfbehrens and
pablo-schmeiser
as code owners
Contributor
Author
|
closes #36 |
pablo-schmeiser
approved these changes
Mar 15, 2026
Collaborator
pablo-schmeiser
left a comment
pablo-schmeiser
left a comment
There was a problem hiding this comment.
Very important and awaited feature!
Looks good to me 👍
Once @hd1ex approves this too, this is ready to be merged.
If @lewellien is still interested in this topic: They may want to review this as well.
Open
Collaborator
|
Thanks for working on iCal support! My first question is: Does this also fix or works upon fixing #41? Can you split the OIDC stuff to another change? I'd rather review and integrate these two things step by step. |
Move _populate_user from AuthorizeSSOUser view into accounts/oidc.py as populate_user_from_oidc so it can be reused by the offline token refresh logic without a model-to-view circular dependency. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
schliz
force-pushed
the
feat/ical-subscription
branch
from
e0f7a40 to
8eb1778
Compare
schliz
changed the title
CalendarToken stores a unique per-user token for session-less iCal feed access. OIDCOfflineToken stores the OIDC refresh token and implements refresh_user_info() to sync groups from the identity provider before serving calendar data. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Introduce TokenAuthMixin that authenticates feed requests via
a CalendarToken URL parameter and refreshes OIDC user data
with a 10-minute cooldown. Register token feed URLs under
/calendar/token/<token>/{user,participation,organization,event}/.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add token management views (create/regenerate/delete) and a Calendar Subscriptions card on the profile page showing copyable iCal URLs with regenerate and revoke controls. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Move the Calendar Subscriptions card from the left sidebar into the right column above Upcoming Shifts for better visibility. Add "Add to Calendar" buttons using the webcal:// protocol so users can subscribe with a single click from their device. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Combine the "Add to Calendar" button and the copyable URL input into one flex row per feed for a more compact layout. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Use the same card structure (p-3, center-items header row) as the user info card. Move regenerate and revoke buttons into the title row as icon-only buttons, and show the generate button there when no token exists. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add German translations for all calendar subscription strings. Rename "My Participations" to "My Shifts" for clarity. Increase spacing between subtitle text and feed sections. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Remove all obsolete (#~) entries including calendar-related strings and fix duplicate "My Shifts" entries that caused msgmerge errors. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
schliz
force-pushed
the
feat/ical-subscription
branch
from
8eb1778 to
11c13e0
Compare
Contributor
Author
schliz
force-pushed
the
feat/ical-subscription
branch
from
3691cff to
11c13e0
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Create a system for persisting
offline_tokens for users that logged in via OpenID Connect. We can use this to periodically update the groups for users that haven't signed in for a while.Resolves #41