Skip to content

Limit atom feed to visible events - #5472

Merged
ThiefMaster merged 1 commit into
indico:masterfrom
bpedersen2:atom_visibility
Mar 15, 2024
Merged

ThiefMaster merged 1 commit into
indico:masterfrom
bpedersen2:atom_visibility

Conversation

@bpedersen2

@bpedersen2 bpedersen2 commented Aug 26, 2022 •

Copy link
Copy Markdown
Contributor

Only export events visible in a category by the atom feed to avoid
unnecessary exposure, especially in the case of exporting to public
displays.

See also[1].

[1] https://talk.getindico.io/t/atom-feeds-should-they-respect-the-visibility-settings/2776

@bpedersen2

Copy link
Copy Markdown
Contributor Author

Ping...

Would be nice to see this in 3.3

@ThiefMaster ThiefMaster added this to the v3.3 milestone Mar 14, 2024
@ThiefMaster

ThiefMaster commented Mar 14, 2024 •

Copy link
Copy Markdown
Member

Thanks for the reminder, I also saw it recently as well when going through the PRs. I'll merge it for 3.3.

Only export events visible in a category by the atom feed to avoid
unnecesary exposure, especially in the case of exporting to public
displays.

See also[1].

[1] https://talk.getindico.io/t/atom-feeds-should-they-respect-the-visibility-settings/2776
@ThiefMaster
ThiefMaster enabled auto-merge (squash) March 15, 2024 17:00
@ThiefMaster
ThiefMaster merged commit 0feba94 into indico:master Mar 15, 2024
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