Skip to content

admin: change default # of results from 10 to 20 #728

admin: change default # of results from 10 to 20

admin: change default # of results from 10 to 20 #728

Triggered via pull request November 1, 2024 13:41
Status Failure
Total duration 2m 5s
Artifacts

tests.yml

on: pull_request
Matrix: JS / Tests
Matrix: Python / Tests
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 20 warnings
Python / Tests (3.9, postgresql14, opensearch2): invenio_administration/views/base.py#L1
Black format check --- /home/runner/work/invenio-administration/invenio-administration/invenio_administration/views/base.py 2024-11-01 13:41:57.476347+00:00 +++ /home/runner/work/invenio-administration/invenio-administration/invenio_administration/views/base.py 2024-11-01 13:43:45.993685+00:00 @@ -371,11 +371,11 @@ available_facets=current_app.config.get(self.search_facets_config_name), sort_options=current_app.config[self.search_sort_config_name], endpoint=self.get_api_endpoint(**kwargs), headers=self.get_search_request_headers(**kwargs), pagination_options=(20, 50), - default_size=20 + default_size=20, ) def get_sort_options(self): """Get search sort options.""" if not self.sort_options:
Python / Tests (3.9, postgresql14, opensearch2)
Process completed with exit code 1.
Python / Tests (3.12, postgresql14, opensearch2): invenio_administration/views/base.py#L1
Black format check --- /home/runner/work/invenio-administration/invenio-administration/invenio_administration/views/base.py 2024-11-01 13:41:57.764902+00:00 +++ /home/runner/work/invenio-administration/invenio-administration/invenio_administration/views/base.py 2024-11-01 13:43:50.224443+00:00 @@ -371,11 +371,11 @@ available_facets=current_app.config.get(self.search_facets_config_name), sort_options=current_app.config[self.search_sort_config_name], endpoint=self.get_api_endpoint(**kwargs), headers=self.get_search_request_headers(**kwargs), pagination_options=(20, 50), - default_size=20 + default_size=20, ) def get_sort_options(self): """Get search sort options.""" if not self.sort_options:
Python / Tests (3.12, postgresql14, opensearch2)
Process completed with exit code 1.
JS / Tests (20.x): invenio_administration/assets/semantic-ui/js/invenio_administration/src/formik/GenerateForm.js#L35
propType "formikProps" is not required, but has no corresponding defaultProps declaration
JS / Tests (20.x): invenio_administration/assets/semantic-ui/js/invenio_administration/src/formik/GenerateForm.js#L37
propType "formData" is not required, but has no corresponding defaultProps declaration
JS / Tests (20.x): invenio_administration/assets/semantic-ui/js/invenio_administration/src/formik/LazyForm.js#L18
handleFieldValueChange should be placed after componentDidUpdate
JS / Tests (20.x): invenio_administration/assets/semantic-ui/js/invenio_administration/src/formik/LazyForm.js#L77
propType "formData" is not required, but has no corresponding defaultProps declaration
JS / Tests (20.x): invenio_administration/assets/semantic-ui/js/invenio_administration/src/formik/fields/fields.js#L50
Fragments should contain more than one child - otherwise, there’s no need for a Fragment at all
JS / Tests (18.x): invenio_administration/assets/semantic-ui/js/invenio_administration/src/formik/GenerateForm.js#L35
propType "formikProps" is not required, but has no corresponding defaultProps declaration
JS / Tests (18.x): invenio_administration/assets/semantic-ui/js/invenio_administration/src/formik/GenerateForm.js#L37
propType "formData" is not required, but has no corresponding defaultProps declaration
JS / Tests (18.x): invenio_administration/assets/semantic-ui/js/invenio_administration/src/formik/LazyForm.js#L18
handleFieldValueChange should be placed after componentDidUpdate
JS / Tests (18.x): invenio_administration/assets/semantic-ui/js/invenio_administration/src/formik/LazyForm.js#L77
propType "formData" is not required, but has no corresponding defaultProps declaration
JS / Tests (18.x): invenio_administration/assets/semantic-ui/js/invenio_administration/src/formik/fields/fields.js#L50
Fragments should contain more than one child - otherwise, there’s no need for a Fragment at all