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.

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

Add Shadow Locale to Settings Tab #238

Merged
merged 2 commits into from
Mar 23, 2023

Conversation

alexander-schranz
Copy link
Member

@alexander-schranz alexander-schranz commented Mar 21, 2023

To provide a shadow locale feature we add a new ShadowInterface for this which provides this feature. Then new shadowLocale and shadowLocales feature is added to the DimensionContent entities and added to the Settings tab.

In upcoming Pull Request the shadow behaviour while publishing should be implemented and checked how we handle the url here.

@alexander-schranz alexander-schranz changed the title Add Shadow Locale to Settings Tab WIP: Add Shadow Locale to Settings Tab Mar 21, 2023
@alexander-schranz alexander-schranz added the Feature New functionality not yet included in Bundle label Mar 21, 2023
@alexander-schranz alexander-schranz force-pushed the feature/shadow-locale branch 3 times, most recently from e2c6502 to 44e8188 Compare March 21, 2023 18:50
@coveralls
Copy link

coveralls commented Mar 21, 2023

Pull Request Test Coverage Report for Build 4498473987

  • 43 of 43 (100.0%) changed or added relevant lines in 6 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 4477935009: 0.0%
Covered Lines: 2434
Relevant Lines: 2434

💛 - Coveralls

@alexander-schranz alexander-schranz force-pushed the feature/shadow-locale branch 7 times, most recently from d2309a3 to 3b1e113 Compare March 21, 2023 19:45
@alexander-schranz alexander-schranz marked this pull request as ready for review March 22, 2023 23:47
@alexander-schranz alexander-schranz changed the title WIP: Add Shadow Locale to Settings Tab Add Shadow Locale to Settings Tab Mar 22, 2023

$normalizedData['shadowOn'] = null !== $object->getShadowLocale();
$normalizedData['shadowLocales'] = $normalizedData['shadowLocales'] ?? [];
$normalizedData['contentLocales'] = $object->getAvailableLocales() ?? []; // TODO should be changed in Sulu Core (PageSettingsShadowLocaleSelect.js)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Resources/translations/admin.de.json Outdated Show resolved Hide resolved
Resources/translations/admin.de.json Outdated Show resolved Hide resolved
Co-authored-by: Prokyonn <daniel.mathis@sector8.eu>
@Prokyonn Prokyonn merged commit 29f27a7 into sulu:0.x Mar 23, 2023
@alexander-schranz alexander-schranz deleted the feature/shadow-locale branch March 23, 2023 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New functionality not yet included in Bundle
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants