Skip to content
Snippets Groups Projects

Add "enable RSS" user privacy toggle (resolves #826)

Merged Hazelnoot requested to merge fEmber/Sharkey:hazelnoot/826-disable-rss into develop
All threads resolved!

What does this MR do?

Adds a user privacy toggle to enable or disable RSS feeds for the user's profile. Feeds are off by default for new users, on by default for existing users (to preserve any existing feeds), and off by default for remote users. The flag is federated via Person.enableRss, although no other software understands this property. This applies equally to the .rss, .atom, and .json feeds.

Contribution Guidelines

By submitting this merge request, you agree to follow our Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines
  • I have made sure to test this merge request

Merge request reports

Merge request pipeline #2096 passed

Merge request pipeline passed for 34c5a3a3

Approval is optional

Merged by HazelnootHazelnoot 1 week ago (Dec 12, 2024 5:34pm UTC)

Merge details

Pipeline #2106 passed

Pipeline passed for d18317a7 on develop

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Marie
  • Hazelnoot added 3 commits

    added 3 commits

    • 65b82d52 - federate profile when changing `enableRss` value
    • bfac371f - safer typings for `userNeedsPublishing` and `profileNeedsPublishing`
    • 622c39ee - fix declaration of `User.enableRss` property

    Compare with previous version

  • Hazelnoot resolved all threads

    resolved all threads

  • Hazelnoot added 1 commit

    added 1 commit

    Compare with previous version

  • Hazelnoot resolved all threads

    resolved all threads

  • Hazelnoot added 37 commits

    added 37 commits

    • 1d236410...943c6414 - 32 commits from branch TransFem-org:develop
    • fe37aa2c - Add "enable RSS" user privacy toggle
    • 02b600c9 - federate profile when changing `enableRss` value
    • 1c65f234 - safer typings for `userNeedsPublishing` and `profileNeedsPublishing`
    • d95ec3de - fix declaration of `User.enableRss` property
    • 34c5a3a3 - fix WebhookTestService.ts

    Compare with previous version

  • dakkar approved this merge request

    approved this merge request

  • Hazelnoot enabled an automatic merge when all merge checks for 34c5a3a3 pass

    enabled an automatic merge when all merge checks for 34c5a3a3 pass

  • dakkar mentioned in issue #634

    mentioned in issue #634

  • Marie approved this merge request

    approved this merge request

  • Hazelnoot mentioned in commit d18317a7

    mentioned in commit d18317a7

  • merged

  • Please register or sign in to reply
    Loading