Skip to content
Snippets Groups Projects

Split character limits between local and remote notes (resolves #723)

Merged Split character limits between local and remote notes (resolves #723)
fEmber/Sharkey:hazelnoot/723-separate-character-limits into develop
All threads resolved!
Merged Hazelnoot requested to merge fEmber/Sharkey:hazelnoot/723-separate-character-limits into develop
All threads resolved!

What does this PR do?

  • Add new maxRemoteNoteLength config to separately limit the length of notes from remote instances.
  • Add new maxCwLength config to separately limit the length of content warnings.
  • Add new maxRemoteCwLength config to separately limit the length of content warnings from remote instances.
  • Add new maxAltTextLength config to separately limit the length of media descriptions.
  • Add new maxRemoteAltTextLength config to separately limit the length of media descriptions from remote instances.
  • Remove obsolete MAX_NOTE_TEXT_LENGTH, DB_MAX_NOTE_TEXT_LENGTH, and DB_MAX_IMAGE_COMMENT_LENGTH constants.
  • Remove hardcoded OpenAPI limit on CW length.

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
Edited by Hazelnoot

Merge request reports

Merge request pipeline #1638 passed

Stage: test

    Merge request pipeline passed for ca1cdc4e

    Merged by JuliaJulia 4 months ago (Oct 29, 2024 3:04am UTC)

    Merge details

    Pipeline #1654 passed

    Stage: test
      Stage: deploy

        Pipeline passed for 1520bc17 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
      • dakkar
      • Hazelnoot added 18 commits

        added 18 commits

        • f0de0261...72a0f16b - 15 commits from branch TransFem-org:develop
        • 175befe7 - separate character limits for local and remote notes
        • 141abb84 - fix UUID format
        • d30a30a1 - clarify comment about MAX_NOTE_TEXT_LENGTH in tests

        Compare with previous version

        Toggle commit list
      • Hazelnoot resolved all threads

        resolved all threads

      • dakkar approved this merge request

        approved this merge request

      • Hazelnoot added 352 commits

        added 352 commits

        • d30a30a1...ba17776b - 349 commits from branch TransFem-org:develop
        • e35c0f7e - separate character limits for local and remote notes
        • 6f7c8f6c - fix UUID format
        • 87787b60 - clarify comment about MAX_NOTE_TEXT_LENGTH in tests

        Compare with previous version

        Toggle commit list
      • Hazelnoot reset approvals from @dakkar by pushing to the branch

        reset approvals from @dakkar by pushing to the branch

      • Hazelnoot added 1 commit

        added 1 commit

        Compare with previous version

      • dakkar approved this merge request

        approved this merge request

      • Charlotte
      • Charlotte
        Charlotte @CenTdemeern1 ·
        Developer

        (I would've left an approval review if I could)

      • Hazelnoot added 62 commits

        added 62 commits

        Compare with previous version

        Toggle commit list
      • Hazelnoot reset approvals from @dakkar by pushing to the branch

        reset approvals from @dakkar by pushing to the branch

      • Hazelnoot resolved all threads

        resolved all threads

      • Hazelnoot added 3 commits

        added 3 commits

        • c5d9bde4 - expose CW limit to frontend
        • 72601305 - show separate counters for text limit and CW limit
        • ca1cdc4e - fix poll option limit in masto API

        Compare with previous version

      • Hazelnoot changed the description

        changed the description

      • Hazelnoot marked this merge request as ready

        marked this merge request as ready

      • Hazelnoot marked the checklist item I have made sure to test this pull request as completed

        marked the checklist item I have made sure to test this pull request as completed

      • dakkar approved this merge request

        approved this merge request

      • dakkar added for-next-release label

        added for-next-release label

      • Julia approved this merge request

        approved this merge request

      • Julia merged

        merged

      • Julia mentioned in commit 1520bc17

        mentioned in commit 1520bc17

      • Hazelnoot added federation label

        added federation label

        • Please register or sign in to reply