Domain Access is currently adding 2 fields (domains and all affiliates) on Node and User.
It should be very interesting to add such restriction on menu entries to allow visibility only for some domains.

CommentFileSizeAuthor
#8 domain-add_domain_access_support_on_menu-2833466-D8-8.patch18.52 KBgarnett2125
#7 domain-add_domain_access_support_on_menu-2833466-D8-7.patch18.53 KBgarnett2125
#2 domain-add_domain_access_support_on_menu-2833466-D8-2.patch19.32 KBSebastien M.
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sebastien @Actualys created an issue. See original summary.

Sebastien M.’s picture

Sebastien M. CreditAttribution: Sebastien M. as a volunteer commented
FileSize
domain-add_domain_access_support_on_menu-2833466-D8-2.patch19.32 KB

Here is my patch.
Hope it helps

Sebastien M.’s picture

Sebastien M. CreditAttribution: Sebastien M. as a volunteer commented
Status: Active » Needs review
agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented

That would have to be maintained as a new, separate module.

Sebastien M.’s picture

Sebastien M. CreditAttribution: Sebastien M. as a volunteer commented

Ok, I'll work on that way.
Bye

Sebastien M.’s picture

Sebastien M. CreditAttribution: Sebastien M. as a volunteer commented

Keep this issue opened, I'll close it once a separate module will be available.
Thanks

garnett2125’s picture

garnett2125 CreditAttribution: garnett2125 commented
FileSize
domain-add_domain_access_support_on_menu-2833466-D8-7.patch18.53 KB

Here is a patch with a domain submodule called Domain Menu Access.

garnett2125’s picture

garnett2125 CreditAttribution: garnett2125 commented
FileSize
domain-add_domain_access_support_on_menu-2833466-D8-8.patch18.52 KB

Removed debug function.

garnett2125’s picture

garnett2125 CreditAttribution: garnett2125 commented

A module is now available instead of using previous patch: https://www.drupal.org/project/domain_menu_access

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented 
+++ b/domain_menu_access/config/install/field.storage.menu_link_content.field_domain_access.yml
@@ -0,0 +1,19 @@
+dependencies:
+  module:
+    - domain
+    - menu_link_content

+++ b/domain_menu_access/config/install/field.storage.menu_link_content.field_domain_all_affiliates.yml
@@ -0,0 +1,18 @@
+dependencies:
+  module:
+    - domain
+    - menu_link_content

both field storages needs to define enforced dependencies for the module see https://github.com/agentrickard/domain/pull/327

zerolab’s picture

zerolab CreditAttribution: zerolab at Torchbox for Mencap commented
Status: Needs review » Closed (works as designed)

That is looking quite good!

Closing this as the functionality is provided by the separate module. Patches/followup should happen in the domain_menu_access issue queue.