Fix test suite's field "slapos parameters" line length
The my_cluster_configuration field was transformed into a proxy field.
The target is a new field with id
my_view_mode_json_text_area (following the base as email and phone fields). It includes JSON validator and the error message for invalid JSON from old my_cluster_configuration field.
Waiting for test results in Nexedi's ERP5.
@seb: since you suggested the proxy field, can you review, please?
Ready to be merged automatically
Ask someone with write access to this repository to merge this request.
Marked this merge request as a Work In ProgressToggle commit list
2 points :
- When moving any object from one business template to another, please do it in a single commit. This is required to keep the history (git automatically detect moves if they are in a single commit)
- Like explained in https://www.erp5.com/documentation/developer/guideline/erp5-Guideline.Form.Profixification, we should have :
erp5_test_result/TestSuite_view/my_cluster_configuration -> erp5_test_result/Base_viewTestResultFieldLibrary/my_view_mode_cluster_configuration -> erp5_core/Base_viewFieldLibrary/my_view_mode_json_text_area
So what you added in erp5_base should go in erp5_core. Indeed, we might need a json_text_area on a business templates which does not have erp5_base in the list of dependencies.
Unmarked this merge request as a Work In ProgressToggle commit list
I tried this commit, it looks good to me. I think let's apply the commit, is it OK ?
don't apply with "Apply patch" button, because it will apply a new commit using the merge request message instead of the original commit (because there's only one commit in this branch)