Skip to content
Snippets Groups Projects

fix: send MFM payload for notes that don't contain advanced MFM

Merged Laura Hausmann requested to merge zotan/Sharkey:fix/mfm-payload into develop

What does this PR do? (Please give us a brief description of what this PR does.) Reverts to the old behavior, where MFM payloads were sent regardless of whether they contain advanced MFM. This allows for significantly faster parsing for instances ingesting notes, at the cost of slightly higher network transfer size. I'd argue there's significantly better ways to optimize that, though, e.g. by removing unused inline JSON-LD @context entries.

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 pull request

resolves #647 (closed)

Edited by Laura Hausmann

Merge request reports

Merge request pipeline #1621 passed

Merge request pipeline passed for d52f4748

Approval is optional

Merged by dakkardakkar 5 months ago (Oct 24, 2024 8:10am UTC)

Merge details

Pipeline #1625 passed

Pipeline passed for 4fed3555 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
Please register or sign in to reply
Loading