Add anonymous calendar links with user tokens #36
Conversation
- Add ical_token to User model - Allow ical export in UserFeed with out authentication but using a token as GET parameter - Add UI for (re-)generating calendar tokens in the profile page Viele Grüße der Schliz
| @@ -30,6 +30,7 @@ | |||
| name='password_reset_success'), | |||
| path('calendar/', UserFeed(), name='user_calendar'), | |||
| path('participation_calendar/', OwnShiftsFeed(), name='user_participation_calendar'), | |||
| path('ical_token_reset/', UserRegenerateCalendarTokenView.as_view(), name='user_ical_token_reset') | |||
There was a problem hiding this comment.
Ich fände rengenerate oder renew da besser da es ja kein reset ist
| def all_shifts_url(self): | ||
| git if self.ical_token is None: | ||
| return None | ||
| return local_settings.SITE + '/user/calendar?token=' + self.ical_token |
There was a problem hiding this comment.
Was tut das?
Einfach Dinge aus local_settings importieren bringt nix das gibts bei dir aber das wars
Mal abgesehen davon das SITE vom Request abhängt
Ein fstring ist mMn ebenfalls wesentlich lesbarer:
f'{SITE}/user../{self.ical_token}'
| def my_shifts_url(self): | ||
| if self.ical_token is None: | ||
| return None | ||
| return local_settings.SITE + '/user/participation_calendar?token=' + self.ical_token |
| <h4>{% trans "Calendar URL" %}</h4> | ||
| <button class="btn btn-info" title="{% trans "Reset iCalendar Link Token" %}" data-bs-toggle="modal" | ||
| data-bs-target="#resetCalendarToken"> | ||
| {% trans "Reset Token" %} <i class="fa-solid fa-repeat"></i> |
| if request.user.is_authenticated: | ||
| return request.user | ||
|
|
||
| # noinspection all |
There was a problem hiding this comment.
Sowas würde ich eher nicht verwenden meistens hat es einen Grund wenn die inspection meckert
| return request.user | ||
|
|
||
| # noinspection all | ||
| user = User.objects.get(ical_token=request.GET.get('token', '')) |
There was a problem hiding this comment.
Das reicht nicht ganz aus, vor allem für SSO-User müsste man das einmal im Monat checken ob der die Gruppen überhaupt noch sehen darf und dazu bräuchte man nen SSO Login.
| @@ -0,0 +1,7 @@ | |||
| <button onclick=" | |||
There was a problem hiding this comment.
Format
Was tut dieses
Auch wenn das inline im Button geht wäre es vmtl besser das irgendwo als inkludierbares Skript schnipsel zu haben.
Viele Grüße der Schliz