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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport 'Nullable component setting generates error :comments_max_length' to v0.27 #11239

Merged

Conversation

alecslupu
Copy link
Contributor

馃帺 What? Why?

Backport #10561 to v0.27

鈾ワ笍 Thank you!

@alecslupu alecslupu added this to the 0.27.4 milestone Jul 19, 2023
@alecslupu alecslupu force-pushed the backport/0.27/nullable-component-setting-gen-10561 branch from b29c257 to 18f4682 Compare July 25, 2023 13:57
@alecslupu alecslupu requested a review from a team July 25, 2023 18:11
@alecslupu alecslupu modified the milestones: 0.27.4, 0.27.5 Jul 26, 2023
alecslupu and others added 2 commits July 31, 2023 13:20
* Fix: Empty proposals config limits

* Make comments_max_length mandatory in accountability

Make comments_max_length mandatory in blogs

Add default value for comments_max_length

Make comments_max_length mandatory in debates

Make comments_max_length mandatory in budgets

Make comments_max_length mandatory in meeetings

Make comments_max_length mandatory in sortitions

Running linters

* Change the factory

* Fixing factory

* Remove the default value of comments_max_length

* Add to_i statements and tests

* Mark Comments max length as required in the interface

* Apply suggestions from code review

Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi>

* Add comments_max_length default, set validator to greater_than_or_equal_to

---------

Co-authored-by: Antti Hukkanen <antti.hukkanen@mainiotech.fi>
@alecslupu alecslupu force-pushed the backport/0.27/nullable-component-setting-gen-10561 branch from 18f4682 to 1e86426 Compare July 31, 2023 10:20
@alecslupu alecslupu added this to Pending review from Product in Maintainers via automation Aug 1, 2023
@fblupi fblupi merged commit d726d7d into release/0.27-stable Aug 9, 2023
46 checks passed
@fblupi fblupi deleted the backport/0.27/nullable-component-setting-gen-10561 branch August 9, 2023 10:25
Maintainers automation moved this from Pending review from Product to Done Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants