Skip to content

Ical/atom feed now returning more than 10 elements for groups

oneacik requested to merge oneacik/mobilizon:master into master

Atom/ical feed for group events was paginated with default settings, therefore returing only 10 first items. Currently, I skipped paging only for feed to show all events. I also refactored code from events to be more code reuse friendly.

On this PR I'd like to focus:

  • is this change acceptable
  • could the code be refactored better?
  • is it readable?
Edited by oneacik

Merge request reports