From 2c79f98c5fe47108c7924ffba47aa1725ad35762 Mon Sep 17 00:00:00 2001 From: DMinnai Date: Tue, 30 May 2023 12:23:30 +0200 Subject: [PATCH 1/3] ENG-4890-amend page repositioning --- src/locales/en.js | 1 + src/locales/it.js | 1 + src/locales/pt.js | 1 + src/state/pages/actions.js | 6 ++++++ 4 files changed, 9 insertions(+) diff --git a/src/locales/en.js b/src/locales/en.js index 9a508ecb6..63e5cf3e9 100644 --- a/src/locales/en.js +++ b/src/locales/en.js @@ -1602,6 +1602,7 @@ export default { 'componentRepository.components.installedVersion': 'Installed Version', 'componentRepository.components.genericError': 'An Error has occurred in the Application', 'page.invalidPositionError': 'A page can only be a direct child of a page with the same owner group or free access', + 'page.invalidChildPositionError': 'A page that belongs to a group cannot become a direct child of the Virtual Root', 'app.installFailed': 'An Error occurred and the Bundle could not be installed', 'app.uninstallFailed': 'An Error occurred and the Bundle could not be uninstalled', 'app.fetchBundlesFailed': 'Failed to fetch bundles from Registry', diff --git a/src/locales/it.js b/src/locales/it.js index 9ab892b8a..9054a6a6d 100644 --- a/src/locales/it.js +++ b/src/locales/it.js @@ -1602,6 +1602,7 @@ export default { 'componentRepository.components.installedVersion': 'Versione installata', 'componentRepository.components.genericError': 'Si è verificato un errore nell\'applicazione', 'page.invalidPositionError': 'Una pagina può essere figlia solo di un\'altra pagina con lo stesso owner group o con owner group free access', + 'page.invalidChildPositionError': 'Una pagina figlia appartenente ad un gruppo non può essere posizionata figlia della root', 'app.installFailed': 'Si è verificato un errore e non è stato possibile installare il Bundle', 'app.uninstallFailed': 'Si è verificato un errore e non è stato possibile disinstallare il Bundle', 'app.fetchBundlesFailed': 'Non è stato possibile recuperare la lista di Bundle dal Registry', diff --git a/src/locales/pt.js b/src/locales/pt.js index 65992fff9..dfa8469bb 100644 --- a/src/locales/pt.js +++ b/src/locales/pt.js @@ -1022,6 +1022,7 @@ export default { 'componentRepository.components.installedVersion': 'Versão Instalada', 'componentRepository.components.genericError': 'Ocorreu um erro no aplicativo', 'page.invalidPositionError': 'Uma página só pode ser filha direta de uma página com o mesmo grupo de proprietários ou free access', + 'page.invalidChildPositionError': 'LIlli', 'app.installFailed': 'Ocorreu um erro e o pacote não pôde ser instalado', 'app.uninstallFailed': 'Ocorreu um erro e o pacote não pôde ser desinstalado', 'app.fetchBundlesFailed': 'Falha ao buscar pacotes do Registro', diff --git a/src/state/pages/actions.js b/src/state/pages/actions.js index 3d70286ae..b356d4b05 100644 --- a/src/state/pages/actions.js +++ b/src/state/pages/actions.js @@ -37,6 +37,9 @@ import { getAppTourProgress } from 'state/app-tour/selectors'; const INVALID_PAGE_POSITION_ERROR = { id: 'page.invalidPositionError' }; const INVALID_PAGE_POSITION_STATUS_CODE = 422; +const INVALID_PAGE_CHILD_POSITION_ERROR = { id: 'page.invalidChildPositionError' }; +const INVALID_PAGE_CHILD_POSITION_STATUS_CODE = 400; + const RESET_FOR_CLONE = { code: '', titles: '', @@ -286,6 +289,9 @@ export const setPageParent = (pageCode, newParentCode) => (dispatch, getState) = dispatch(setPageParentSync(pageCode, oldParentCode, newParentCode)); } else if (response && response.status === INVALID_PAGE_POSITION_STATUS_CODE) { dispatch(addToast(INVALID_PAGE_POSITION_ERROR, TOAST_ERROR)); + } else if (response && response.status === INVALID_PAGE_CHILD_POSITION_STATUS_CODE) { + console.log('lilli', response.status); + dispatch(addToast(INVALID_PAGE_CHILD_POSITION_ERROR, TOAST_ERROR)); } else { response.json().then((json) => { json.errors.forEach(err => dispatch(addToast(err.message, TOAST_ERROR))); From 59f31f82894d1817845d633ce4f17806f766e6ca Mon Sep 17 00:00:00 2001 From: DMinnai Date: Tue, 30 May 2023 13:12:21 +0200 Subject: [PATCH 2/3] ENG-4890 localization added in case of 400 bad request --- src/locales/en.js | 2 +- src/locales/it.js | 2 +- src/locales/pt.js | 2 +- src/state/pages/actions.js | 1 - 4 files changed, 3 insertions(+), 4 deletions(-) diff --git a/src/locales/en.js b/src/locales/en.js index 63e5cf3e9..57c17e832 100644 --- a/src/locales/en.js +++ b/src/locales/en.js @@ -1602,7 +1602,7 @@ export default { 'componentRepository.components.installedVersion': 'Installed Version', 'componentRepository.components.genericError': 'An Error has occurred in the Application', 'page.invalidPositionError': 'A page can only be a direct child of a page with the same owner group or free access', - 'page.invalidChildPositionError': 'A page that belongs to a group cannot become a direct child of the Virtual Root', + 'page.invalidChildPositionError': 'A page belonging to a group cannot be placed as a child of the root', 'app.installFailed': 'An Error occurred and the Bundle could not be installed', 'app.uninstallFailed': 'An Error occurred and the Bundle could not be uninstalled', 'app.fetchBundlesFailed': 'Failed to fetch bundles from Registry', diff --git a/src/locales/it.js b/src/locales/it.js index 9054a6a6d..7ca10eda6 100644 --- a/src/locales/it.js +++ b/src/locales/it.js @@ -1602,7 +1602,7 @@ export default { 'componentRepository.components.installedVersion': 'Versione installata', 'componentRepository.components.genericError': 'Si è verificato un errore nell\'applicazione', 'page.invalidPositionError': 'Una pagina può essere figlia solo di un\'altra pagina con lo stesso owner group o con owner group free access', - 'page.invalidChildPositionError': 'Una pagina figlia appartenente ad un gruppo non può essere posizionata figlia della root', + 'page.invalidChildPositionError': 'Una pagina appartenente ad un gruppo non può essere posizionata figlia della root', 'app.installFailed': 'Si è verificato un errore e non è stato possibile installare il Bundle', 'app.uninstallFailed': 'Si è verificato un errore e non è stato possibile disinstallare il Bundle', 'app.fetchBundlesFailed': 'Non è stato possibile recuperare la lista di Bundle dal Registry', diff --git a/src/locales/pt.js b/src/locales/pt.js index dfa8469bb..0001ebdb3 100644 --- a/src/locales/pt.js +++ b/src/locales/pt.js @@ -1022,7 +1022,7 @@ export default { 'componentRepository.components.installedVersion': 'Versão Instalada', 'componentRepository.components.genericError': 'Ocorreu um erro no aplicativo', 'page.invalidPositionError': 'Uma página só pode ser filha direta de uma página com o mesmo grupo de proprietários ou free access', - 'page.invalidChildPositionError': 'LIlli', + 'page.invalidChildPositionError': 'Uma página que pertence a um grupo não pode se tornar um filho direto da raiz atual', 'app.installFailed': 'Ocorreu um erro e o pacote não pôde ser instalado', 'app.uninstallFailed': 'Ocorreu um erro e o pacote não pôde ser desinstalado', 'app.fetchBundlesFailed': 'Falha ao buscar pacotes do Registro', diff --git a/src/state/pages/actions.js b/src/state/pages/actions.js index b356d4b05..cb088660c 100644 --- a/src/state/pages/actions.js +++ b/src/state/pages/actions.js @@ -290,7 +290,6 @@ export const setPageParent = (pageCode, newParentCode) => (dispatch, getState) = } else if (response && response.status === INVALID_PAGE_POSITION_STATUS_CODE) { dispatch(addToast(INVALID_PAGE_POSITION_ERROR, TOAST_ERROR)); } else if (response && response.status === INVALID_PAGE_CHILD_POSITION_STATUS_CODE) { - console.log('lilli', response.status); dispatch(addToast(INVALID_PAGE_CHILD_POSITION_ERROR, TOAST_ERROR)); } else { response.json().then((json) => { From 9feed52457f8334b68fc470300cf02a60e9792ab Mon Sep 17 00:00:00 2001 From: DMinnai Date: Tue, 30 May 2023 13:48:45 +0200 Subject: [PATCH 3/3] ENG-4899 localization added in case of 400 bad request --- src/locales/en.js | 2 +- src/locales/it.js | 2 +- src/locales/pt.js | 2 +- src/state/pages/actions.js | 6 +----- 4 files changed, 4 insertions(+), 8 deletions(-) diff --git a/src/locales/en.js b/src/locales/en.js index 57c17e832..35621724b 100644 --- a/src/locales/en.js +++ b/src/locales/en.js @@ -1601,8 +1601,8 @@ export default { 'componentRepository.categories.group': 'Group', 'componentRepository.components.installedVersion': 'Installed Version', 'componentRepository.components.genericError': 'An Error has occurred in the Application', - 'page.invalidPositionError': 'A page can only be a direct child of a page with the same owner group or free access', 'page.invalidChildPositionError': 'A page belonging to a group cannot be placed as a child of the root', + 'page.invalidPositionError': 'A page can only be a direct child of a page with the same owner group or free access', 'app.installFailed': 'An Error occurred and the Bundle could not be installed', 'app.uninstallFailed': 'An Error occurred and the Bundle could not be uninstalled', 'app.fetchBundlesFailed': 'Failed to fetch bundles from Registry', diff --git a/src/locales/it.js b/src/locales/it.js index 7ca10eda6..f0aaaab34 100644 --- a/src/locales/it.js +++ b/src/locales/it.js @@ -1601,8 +1601,8 @@ export default { 'componentRepository.categories.group': 'Gruppo', 'componentRepository.components.installedVersion': 'Versione installata', 'componentRepository.components.genericError': 'Si è verificato un errore nell\'applicazione', - 'page.invalidPositionError': 'Una pagina può essere figlia solo di un\'altra pagina con lo stesso owner group o con owner group free access', 'page.invalidChildPositionError': 'Una pagina appartenente ad un gruppo non può essere posizionata figlia della root', + 'page.invalidPositionError': 'Una pagina può essere figlia solo di un\'altra pagina con lo stesso owner group o con owner group free access', 'app.installFailed': 'Si è verificato un errore e non è stato possibile installare il Bundle', 'app.uninstallFailed': 'Si è verificato un errore e non è stato possibile disinstallare il Bundle', 'app.fetchBundlesFailed': 'Non è stato possibile recuperare la lista di Bundle dal Registry', diff --git a/src/locales/pt.js b/src/locales/pt.js index 0001ebdb3..a54541dff 100644 --- a/src/locales/pt.js +++ b/src/locales/pt.js @@ -1021,8 +1021,8 @@ export default { 'componentRepository.categories.group': 'Grupo', 'componentRepository.components.installedVersion': 'Versão Instalada', 'componentRepository.components.genericError': 'Ocorreu um erro no aplicativo', - 'page.invalidPositionError': 'Uma página só pode ser filha direta de uma página com o mesmo grupo de proprietários ou free access', 'page.invalidChildPositionError': 'Uma página que pertence a um grupo não pode se tornar um filho direto da raiz atual', + 'page.invalidPositionError': 'Uma página só pode ser filha direta de uma página com o mesmo grupo de proprietários ou free access', 'app.installFailed': 'Ocorreu um erro e o pacote não pôde ser instalado', 'app.uninstallFailed': 'Ocorreu um erro e o pacote não pôde ser desinstalado', 'app.fetchBundlesFailed': 'Falha ao buscar pacotes do Registro', diff --git a/src/state/pages/actions.js b/src/state/pages/actions.js index cb088660c..aa8068161 100644 --- a/src/state/pages/actions.js +++ b/src/state/pages/actions.js @@ -240,11 +240,7 @@ export const fetchPageTree = pageCode => async (dispatch) => { return response.payload; }; -/** - * will call: - * http://confluence.entando.org/display/E5/Page+Tree - * /pages - */ + export const handleExpandPage = (pageCode = HOMEPAGE_CODE, alwaysExpand) => ( (dispatch, getState) => { const state = getState();