Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MAX_POST_CHARS Env #27629

Closed
wants to merge 15 commits into from
Closed

MAX_POST_CHARS Env #27629

wants to merge 15 commits into from

Conversation

shleeable
Copy link
Contributor

@shleeable shleeable commented Oct 31, 2023

I completed a quick survey of 10,000+ Mastodon instances and found that 21% of them change their MAX_POST length in some way... that's more than 2100 instance admins messing with the default.... and causing problems/extra work for themselves during upgrades with pointless merge conflicts (for non-glitch admins).

Results here: https://gist.github.com/shleeable/101b131653c0ff0e21a81bf99a31c7f0

As such, I've stolen the code from Glitch for MAX_TOOT_CHARS.... @ClearlyClaire please make sure I'm not missing anything

RIP TOOT

@shleeable
Copy link
Contributor Author

Example of the survey

   10335 | TOTAL LINES
     140 | DISTINCT LINES

COUNT    | LINE
-----------------------------------------------------
    8138 | 500
     942 | 5000
     445 | 3000
     135 | 1000
      81 | 10000
      77 | 2000
      49 | 1500
      35 | 100000

@shleeable
Copy link
Contributor Author

@ClearlyClaire updated.

@vmstan
Copy link
Contributor

vmstan commented Nov 28, 2023

Fixes #12265

@vmstan
Copy link
Contributor

vmstan commented Dec 14, 2023

I wonder why the rebase required hasn't triggered for this? The initializer file was redone in #28363 and the patch won't apply anymore.

@shleeable
Copy link
Contributor Author

Copy link

codecov bot commented Dec 17, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (da72908) 84.79% compared to head (74da98d) 84.79%.
Report is 224 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #27629   +/-   ##
=======================================
  Coverage   84.79%   84.79%           
=======================================
  Files        1039     1039           
  Lines       28253    28255    +2     
  Branches     4557     4557           
=======================================
+ Hits        23956    23958    +2     
  Misses       3139     3139           
  Partials     1158     1158           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@adulau
Copy link

adulau commented Dec 24, 2023

Can we expect a merge for the release 4.2.4 ? This would be awesome.

@ClearlyClaire
Copy link
Contributor

I don't think @Gargron's stance against making the character limit customizable has changed.

@jgillich
Copy link
Contributor

jgillich commented Jan 6, 2024

OP does make a strong case for this change in my opinion: 21% of admins already jump through the hoops to change this. It is a frequent request by users, and most other fediverse servers support custom character limits. Gargron is against this because the UI/UX is designed for 500 characters, but you could also mention this in the documentation for this option and make it clear that a limit of 500 is the official recommendation. The majority of admins will stick with the default, but you'll remove pain points for those who'll change it anyhow.

@PinballsWizard
Copy link

I would be interested to know if the survey data collected also includes the version each instance is running, as compared to the then-current most up to date version. The additional code changes add complexity to upgrades and simplifying this may be beneficial to ensuring more instance admins update in a more timely manner.

@shleeable
Copy link
Contributor Author

The battle of MAX_CHAR Hill continues.

@nightpool
Copy link
Member

my stance has always been that I think it's much more healthy for the ecosystem for thousands of admins to learn how to make changes and customize the software they run rather than having to shoulder the complexity of infinite possible customizations in thr base code. additionally I mostly agree wrt the UI. By not adding this to the base mastodon codebase, we encourage the development of alternative server software and client UI that's more purpose built for longer posts, like Tumblr.

@shleeable
Copy link
Contributor Author

shleeable commented Jan 8, 2024

we encourage the development of alternative server software and client UI that's more purpose built for longer posts, like Tumblr.

Me too pal.

It's been a dark time for people like me that believe Tumblr is the superior social network.... maybe 2024 is the year of the FediTumblr clone.

@mwt
Copy link

mwt commented Jan 17, 2024

we encourage the development of alternative server software and client UI that's more purpose built for longer posts, like Tumblr.

Me too pal.

It's been a dark time for people like me that believe Tumblr is the superior social network.... maybe 2024 is the year of the FediTumblr clone.

or just fedi support in tumblr, which is likely!

@gabboman
Copy link

we encourage the development of alternative server software and client UI that's more purpose built for longer posts, like Tumblr.

Me too pal.

It's been a dark time for people like me that believe Tumblr is the superior social network.... maybe 2024 is the year of the FediTumblr clone.

wafrn added fedederation in 2023 but maybe 2025 will be the year of wafrn (I am trying to be real)

@cmdr-nova
Copy link

I support a max post toggle, this would be amazing

@henryclw
Copy link

Amazing work on this PR! Really appreciate the effort involved. LGTM (Looks Good To Me) and excited to see it merged. 👍

Copy link
Contributor

This pull request has merge conflicts that must be resolved before it can be merged.

@adulau
Copy link

adulau commented Jan 28, 2024

Can we expect a merge for the release 4.2.5 ? This would be awesome. I must admit it will be soon my trigger to move to a fork if it's not finally in Mastodon....

@arazilsongweaver
Copy link

Most of the UI/UX functionality to support this feature was recently merged in via pull request #28928 ... AFAIK the only thing left to do is to implement the server side of things and clean up any resulting QA/QC items.

@shleeable shleeable closed this Jan 29, 2024
@shleeable shleeable deleted the patch-1 branch January 29, 2024 10:55
@henryclw
Copy link

Any update of this request? @shleeable May I ask if this merge is closed just because that a newer merge is created or other should take over of this and create a new merge?

@rahilarious
Copy link

Instead of being dictated by one guy and his decision, Mastodon should use it's own feature and take a poll to make decision.

I shouldn't be needing to compile from source or use fork just for one simple env variable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet