@plone/volto 16.5.0 → 16.6.0

package/.changelog.draft

@@ -1,15 +1,18 @@
-## 16.5.0 (2022-12-16)
+## 16.6.0 (2022-12-23)
 
 ### Feature
 
-- Update pt_BR translation @rafahela [#4121](https://github.com/plone/volto/issues/4121)
+- Allow passing the `step` prop to NumberWidget @tiberiuichim [#4152](https://github.com/plone/volto/issues/4152)
 
 ### Bugfix
 
-- Update SlashMenu to include block title in shortcut search; and prevent default on keydown for Arrows and Enter @danalvrz [#4116](https://github.com/plone/volto/issues/4116)
-- Fix removing of toast notification of logout when user login once again. @iFlameing [#4125](https://github.com/plone/volto/issues/4125)
+- Fix categorization list is not readable when there are longer values or filtering @iFlameing [#4113](https://github.com/plone/volto/issues/4113)
+- Add scroll into view settings @robgietema [#4140](https://github.com/plone/volto/issues/4140)
+- Remove searching capabilities of SortOn component of Search Block @iFlameing [#4162](https://github.com/plone/volto/issues/4162)
+- Fixed externalRoutes short version @pnicolli [#4182](https://github.com/plone/volto/issues/4182)
 
-### Internal
+### Documentation
 
-- Upgrade testbed and convenience api folder to Plone 6 final @sneridagh [#4105](https://github.com/plone/volto/issues/4105)
+- Fix URLs to Plone 6 docs. @stevepiercy [#4143](https://github.com/plone/volto/issues/4143)
+- Fix Sphinx warning Document headings start at H2, not H1 [myst.header]. @stevepiercy [#4145](https://github.com/plone/volto/issues/4145)
 

package/CHANGELOG.md

@@ -8,6 +8,25 @@
 
 <!-- towncrier release notes start -->
 
+## 16.6.0 (2022-12-23)
+
+### Feature
+
+- Allow passing the `step` prop to NumberWidget @tiberiuichim [#4152](https://github.com/plone/volto/issues/4152)
+
+### Bugfix
+
+- Fix categorization list is not readable when there are longer values or filtering @iFlameing [#4113](https://github.com/plone/volto/issues/4113)
+- Add scroll into view settings @robgietema [#4140](https://github.com/plone/volto/issues/4140)
+- Remove searching capabilities of SortOn component of Search Block @iFlameing [#4162](https://github.com/plone/volto/issues/4162)
+- Fixed externalRoutes short version @pnicolli [#4182](https://github.com/plone/volto/issues/4182)
+
+### Documentation
+
+- Fix URLs to Plone 6 docs. @stevepiercy [#4143](https://github.com/plone/volto/issues/4143)
+- Fix Sphinx warning Document headings start at H2, not H1 [myst.header]. @stevepiercy [#4145](https://github.com/plone/volto/issues/4145)
+
+
 ## 16.5.0 (2022-12-16)
 
 ### Feature

package/package.json

@@ -9,7 +9,7 @@
     }
   ],
   "license": "MIT",
-  "version": "16.5.0",
+  "version": "16.6.0",
   "repository": {
     "type": "git",
     "url": "git@github.com:plone/volto.git"

package/packages/volto-slate/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plone/volto-slate",
-  "version": "16.5.0",
+  "version": "16.6.0",
   "description": "Slate.js integration with Volto",
   "main": "src/index.js",
   "author": "European Environment Agency: IDM2 A-Team",

package/packages/volto-slate/src/index.js

@@ -34,6 +34,8 @@ export default (config) => {
     'styleMenu',
   ];
 
+  config.settings.slate.scrollIntoView = false;
+
   config.addonReducers = {
     ...config.addonReducers,
     ...slateReducers,

package/src/components/manage/Blocks/Search/components/SortOn.jsx

@@ -80,6 +80,7 @@ const SortOn = (props) => {
               label: sortable_indexes[k]?.title || k,
             })),
           ]}
+          isSearchable={false}
           value={value}
           onChange={(data) => {
             !isEditMode && setSortOn(data.value);

package/src/components/manage/Widgets/NumberWidget.jsx

@@ -32,6 +32,7 @@ const NumberWidget = (props) => {
     maximum,
     minimum,
     placeholder,
+    step,
   } = props;
 
   return (
@@ -43,6 +44,7 @@ const NumberWidget = (props) => {
         disabled={isDisabled}
         min={minimum || null}
         max={maximum || null}
+        step={step}
         value={value ?? ''}
         placeholder={placeholder}
         onChange={({ target }) =>
@@ -65,7 +67,7 @@ const NumberWidget = (props) => {
 NumberWidget.propTypes = {
   id: PropTypes.string.isRequired,
   title: PropTypes.string.isRequired,
-  description: PropTypes.string,
+  description: PropTypes.oneOfType([PropTypes.string, PropTypes.object]),
   required: PropTypes.bool,
   error: PropTypes.arrayOf(PropTypes.string),
   value: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
@@ -73,6 +75,7 @@ NumberWidget.propTypes = {
   wrapped: PropTypes.bool,
   maximum: PropTypes.number,
   minimum: PropTypes.number,
+  step: PropTypes.number,
   placeholder: PropTypes.string,
 };
 

package/src/routes.js

@@ -2,6 +2,8 @@
  * Routes.
  * @module routes
  */
+import debug from 'debug';
+import { compact } from 'lodash';
 import {
   Add,
   AddonsControlpanel,
@@ -93,12 +95,36 @@ export const multilingualRoutes = [
   },
 ];
 
+export function getExternalRoutes() {
+  return compact(
+    (config.settings?.externalRoutes || []).map((route) => {
+      const newRoute = {
+        component: NotFound,
+      };
+      if (typeof route.match === 'string') {
+        newRoute.path = route.match;
+        return newRoute;
+      } else if (
+        typeof route.match === 'object' &&
+        !Array.isArray(route.match)
+      ) {
+        return {
+          ...newRoute,
+          ...route.match,
+        };
+      } else {
+        debug('routes')(
+          'Got invalid externalRoute, please check the configuration.',
+        );
+        return null;
+      }
+    }),
+  );
+}
+
 export const defaultRoutes = [
   // redirect to external links if path is in blacklist
-  ...(config.settings?.externalRoutes || []).map((route) => ({
-    ...route.match,
-    component: NotFound,
-  })),
+  ...getExternalRoutes(),
   ...((config.settings?.isMultilingual && multilingualRoutes) || []),
   {
     path: '/',

package/src/routes.test.js

@@ -0,0 +1,34 @@
+import config from './registry';
+import { getExternalRoutes } from './routes';
+
+describe('externalRoutes', () => {
+  it('computes regular externalRoutes correctly', () => {
+    config.settings.externalRoutes = [
+      {
+        match: {
+          path: '/test',
+        },
+      },
+    ];
+    const testRoute = getExternalRoutes().find((r) => r.path === '/test');
+    expect(testRoute).not.toBeUndefined();
+  });
+  it('computes shorthand externalRoutes correctly', () => {
+    config.settings.externalRoutes = [{ match: '/test' }];
+    const testRoute = getExternalRoutes().find((r) => r.path === '/test');
+    expect(testRoute).not.toBeUndefined();
+  });
+  it('ignores invalid routes', () => {
+    config.settings.externalRoutes = [
+      '/test',
+      { '/test': true },
+      ['/test'],
+      { match: ['/test'] },
+      { match: 123 },
+    ];
+    const externalRoutes = getExternalRoutes();
+    const testRoute = externalRoutes.find((r) => r.path === '/test');
+    expect(testRoute).toBeUndefined();
+    expect(externalRoutes.length).toEqual(0);
+  });
+});

package/news/4113.bug

@@ -1 +0,0 @@
-Fix categorization list is not readable when there are longer values or filtering @iFlameing