fix: saving of the hostname overrides modal

Author: boazpoolman

.changeset/crazy-seals-go.md

@@ -0,0 +1,5 @@
+---
+"webtools-addon-sitemap": patch
+---
+
+fix: saving of the hostname overrides modal

packages/addons/sitemap/admin/components/HostnameModal/index.jsx

@@ -26,7 +26,7 @@ const ModalForm = (props) => {
 
   useEffect(() => {
     if (isOpen) {
-      setHostnames({ ...hostnameOverrides });
+      setHostnames({ ...hostnameOverrides.toJS() });
     } else {
       setHostnames({});
     }

packages/addons/sitemap/admin/state/reducers/Sitemap/index.js

@@ -69,7 +69,7 @@ export default function sitemapReducer(state = initialState, action) {
         .updateIn(['modifiedCustomEntries', action.url, action.key], () => action.value);
     case ON_CHANGE_SETTINGS:
       return state
-        .updateIn(['settings', 'sitemaps', action.id, action.key], () => action.value);
+        .updateIn(['settings', 'sitemaps', action.id, action.key], () => fromJS(action.value));
     case DISCARD_ALL_CHANGES:
       return state
         .updateIn(['settings', 'sitemaps', action.id], () => state.get('initialData'))

packages/addons/sitemap/admin/tabs/Settings/index.jsx

@@ -24,7 +24,7 @@ const Settings = ({ id }) => {
   const [open, setOpen] = useState(false);
   const languages = useSelector((store) => store.getIn(['sitemap', 'languages'], {}));
   const settings = useSelector((state) => state.getIn(['sitemap', 'settings', 'sitemaps', id], Map()));
-  const hostnameOverrides = useSelector((state) => state.getIn(['sitemap', 'settings', 'sitemaps', id, 'hostname_overrides'], {}));
+  const hostnameOverrides = useSelector((state) => state.getIn(['sitemap', 'settings', 'sitemaps', id, 'hostname_overrides'], Map()));
   const [inputVisible, setInputVisible] = useState(settings.get('defaultLanguageUrlType') === DEFAULT_LANGUAGE_URL_TYPE_OTHER);
 
   const saveHostnameOverrides = (hostnames) => {