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
MAX_POST_CHARS Env #27629
Conversation
Example of the survey
|
@ClearlyClaire updated. |
Fixes #12265 |
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. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
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. |
Can we expect a merge for the release 4.2.4 ? This would be awesome. |
I don't think @Gargron's stance against making the character limit customizable has changed. |
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. |
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. |
The battle of MAX_CHAR Hill continues. |
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. |
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! |
wafrn added fedederation in 2023 but maybe 2025 will be the year of wafrn (I am trying to be real) |
I support a max post toggle, this would be amazing |
Amazing work on this PR! Really appreciate the effort involved. LGTM (Looks Good To Me) and excited to see it merged. 👍 |
This pull request has merge conflicts that must be resolved before it can be merged. |
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.... |
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. |
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? |
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 |
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