Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Added paragraph about siteaccess scopes are merged #2475

Open
wants to merge 3 commits into
base: 4.6
Choose a base branch
from
Open

Added paragraph about siteaccess scopes are merged #2475

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
179ef66
Added paragraph about siteaccess scopes are merged
vidarl Aug 21, 2024
eaeb579
fixup! Added paragraph about siteaccess scopes are merged
vidarl Aug 21, 2024
ce1a1a9
fixup! Added paragraph about siteaccess scopes are merged
vidarl Aug 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions docs/multisite/multisite_configuration.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,20 @@
Identifier\ContentType: [article]
```

Be aware of the fact that configurations in the different scopes are merged, not overwritten. This is important to have

Check warning on line 138 in docs/multisite/multisite_configuration.md

View workflow job for this annotation

GitHub Actions / vale

[vale] docs/multisite/multisite_configuration.md#L138 

[Ibexa.UnclearAntecedent] Instead of 'This is', try starting this sentence with a specific subject and verb.
Raw output 
{"message": "[Ibexa.UnclearAntecedent] Instead of 'This is', try starting this sentence with a specific subject and verb.", "location": {"path": "docs/multisite/multisite_configuration.md", "range": {"start": {"line": 138, "column": 95}}}, "severity": "WARNING"}
in mind when dealing with arrays:

``` yaml
ibexa:
system:
default:
languages: ['eng-GB']
de:
languages: ['ger-DE']
```

The language setting for the `de` siteaccess will in this case be `['ger-DE','eng-GB']`, not `['ger-DE']`.

Check failure on line 150 in docs/multisite/multisite_configuration.md

View workflow job for this annotation

GitHub Actions / vale

[vale] docs/multisite/multisite_configuration.md#L150 

[Ibexa.DomainTermCapitalization] Use 'SiteAccess' instead of 'siteaccess'
Raw output 
{"message": "[Ibexa.DomainTermCapitalization] Use 'SiteAccess' instead of 'siteaccess'", "location": {"path": "docs/multisite/multisite_configuration.md", "range": {"start": {"line": 150, "column": 35}}}, "severity": "ERROR"}

Check warning on line 150 in docs/multisite/multisite_configuration.md

View workflow job for this annotation

GitHub Actions / vale

[vale] docs/multisite/multisite_configuration.md#L150 

[Ibexa.FutureTense] Use present tense instead of future.
Raw output 
{"message": "[Ibexa.FutureTense] Use present tense instead of future.", "location": {"path": "docs/multisite/multisite_configuration.md", "range": {"start": {"line": 150, "column": 46}}}, "severity": "WARNING"}

### SiteAccesses and Page Builder [[% include 'snippets/experience_badge.md' %]] [[% include 'snippets/commerce_badge.md' %]]

To define which SiteAccesses are available in the submenu in Page Builder, use the following configuration:
Expand Down
Loading