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

feat: personalised ical feed #1101

Merged
merged 25 commits into from
Oct 25, 2024
Merged

feat: personalised ical feed #1101

merged 25 commits into from
Oct 25, 2024

Conversation

SilasPeters
Copy link
Member

@SilasPeters SilasPeters commented May 21, 2024

This PR introduced a new button, 'Copy personalised Webcal link`, which copies the link to your ical feed which any calendar client can use to visualise the activities where you are enrolled.

It also adds clarification on the difference between the ical feed and the ical templates

This will close #1037

@SilasPeters SilasPeters added the enhancement Proposes (changes to) a feature label May 21, 2024
@SilasPeters SilasPeters self-assigned this May 21, 2024
@SilasPeters SilasPeters changed the title Feat/personalised ical feed feat: personalised ical feed May 23, 2024
@SilasPeters SilasPeters force-pushed the feat/personalised-ical-feed branch from 3d4b6d5 to 9f1a972 Compare May 25, 2024 12:39
@SilasPeters SilasPeters marked this pull request as ready for review May 25, 2024 12:46
@SilasPeters SilasPeters force-pushed the feat/personalised-ical-feed branch 4 times, most recently from 33f7792 to 7c17300 Compare May 25, 2024 13:39
@SilasPeters SilasPeters marked this pull request as draft May 26, 2024 08:20
Copy link
Contributor

@Siem2l Siem2l left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few comments

app/controllers/api/calendars_controller.rb Outdated Show resolved Hide resolved
app/models/member.rb Outdated Show resolved Hide resolved
config/locales/en.yml Show resolved Hide resolved
config/locales/nl.yml Outdated Show resolved Hide resolved
lib/icalendar_helper.rb Outdated Show resolved Hide resolved
@SilasPeters SilasPeters marked this pull request as ready for review June 4, 2024 10:52
@SilasPeters
Copy link
Member Author

I now realise that there are also activities where you cannot sign up for. This might raise confusion, for you bought a pretix ticket but the activity does not appear in your calendar.

I should add some clarification on what exactly the personalized calendar feed is, or add a checkbox next to every activity to include the activity in your calendar, where it is automatically checked when you sign up.

@SilasPeters SilasPeters force-pushed the feat/personalised-ical-feed branch from b6fd936 to 7f35699 Compare August 2, 2024 16:27
@SilasPeters SilasPeters requested review from Siem2l and stickyPiston and removed request for Siem2l and stickyPiston August 2, 2024 16:28
@SilasPeters SilasPeters force-pushed the feat/personalised-ical-feed branch from 7f35699 to 729f0c3 Compare August 2, 2024 16:31
stickyPiston
stickyPiston previously approved these changes Oct 17, 2024
Copy link
Member

@stickyPiston stickyPiston left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

頑張ったね!

config/locales/en.yml Show resolved Hide resolved
config/locales/members.en.yml Show resolved Hide resolved
config/locales/members.nl.yml Outdated Show resolved Hide resolved
@stickyPiston
Copy link
Member

Also on a separate note: it might be nice to mention how to import the personalised calendar into one's own calendar app somewhere. But i'd love to see this feature get merged as soon as possible first.

Copy link
Member

@stickyPiston stickyPiston left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

また頑張ったね!

@SilasPeters SilasPeters merged commit 01c3d07 into master Oct 25, 2024
1 check passed
@SilasPeters SilasPeters deleted the feat/personalised-ical-feed branch October 25, 2024 13:55
SilasPeters added a commit that referenced this pull request Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Proposes (changes to) a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Generate and maintain user-specific calendars containing the corresponding users' activities
3 participants