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

Add missing doc changes and limit string length 4000 to new columns #31825

Merged
merged 1 commit into from
Apr 7, 2022

Conversation

nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Apr 4, 2022

Follow up to #31679
Also adding #31513 even when disabled since #31580 so apps can keep it in mind for the future

Related nextcloud/documentation#8174

@nickvergessen nickvergessen added this to the Nextcloud 24 milestone Apr 4, 2022
@nickvergessen nickvergessen added bug 3. to review Waiting for reviews labels Apr 4, 2022
@nickvergessen nickvergessen requested review from a team, PVince81, icewind1991 and juliusknorr and removed request for a team April 4, 2022 13:58
@nickvergessen nickvergessen force-pushed the bugfix/noid/add-missing-docs branch from a580fff to 57c9b62 Compare April 4, 2022 14:00
@skjnldsv
Copy link
Member

skjnldsv commented Apr 7, 2022

cc @come-nc

@nickvergessen nickvergessen changed the title Add missing doc changes Add missing doc changes and limit string length 4000 to new columns Apr 7, 2022
@blizzz
Copy link
Member

blizzz commented Apr 7, 2022

superseded by #31872 but the doc parts can be taken over still

Signed-off-by: Joas Schilling <coding@schilljs.com>
@come-nc come-nc force-pushed the bugfix/noid/add-missing-docs branch from 5f1f27b to 10b2319 Compare April 7, 2022 13:51
@come-nc
Copy link
Contributor

come-nc commented Apr 7, 2022

(rebased on master, removed redundant commit, only doc changes left, good to merge)

@blizzz
Copy link
Member

blizzz commented Apr 7, 2022

no code change → merging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants