Skip to content

[FEATURE] manifest.json: Auto-fill supportedLocales#617

Merged
flovogt merged 1 commit intomainfrom
manifest-transformer
Jul 12, 2024
Merged

[FEATURE] manifest.json: Auto-fill supportedLocales#617
flovogt merged 1 commit intomainfrom
manifest-transformer

Conversation

@flovogt
Copy link
Member

@flovogt flovogt commented Nov 20, 2023

d3xter666
d3xter666 previously approved these changes Jan 23, 2024
@d3xter666 d3xter666 dismissed their stale review January 23, 2024 08:39

failing tests

Copy link
Contributor

@d3xter666 d3xter666 left a comment

Choose a reason for hiding this comment

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

LGTM

if (rManifest.test(resourcePath)) {
// Special handling for manifest.json file by adding additional content to the served manifest.json
const {default: manifestEnricher} = await import("@ui5/builder/processors/manifestEnricher");
await manifestEnricher({
Copy link
Member

Choose a reason for hiding this comment

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

The processor modifies the resource (via setString), which can cause some inconsistent behaviour during development regarding caching, so we might need to disable etag caching for manifest.json. I need to check this again to understand when it exactly happens..

@matz3 matz3 force-pushed the manifest-transformer branch from f3bd5eb to ebaad4d Compare March 15, 2024 11:25
@flovogt flovogt force-pushed the manifest-transformer branch 3 times, most recently from 4883586 to 3d426cc Compare March 28, 2024 15:27
@flovogt flovogt force-pushed the manifest-transformer branch from 3d426cc to f9b4585 Compare April 4, 2024 12:11
@flovogt flovogt force-pushed the manifest-transformer branch from f9b4585 to b1a688c Compare April 17, 2024 12:54
@matz3 matz3 force-pushed the manifest-transformer branch from b1a688c to 08a3c28 Compare April 29, 2024 11:59
Copy link
Member

@matz3 matz3 left a comment

Choose a reason for hiding this comment

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

LGTM

@flovogt flovogt force-pushed the manifest-transformer branch 3 times, most recently from ac1bb85 to afc2fde Compare May 27, 2024 12:47
@flovogt flovogt force-pushed the manifest-transformer branch 2 times, most recently from 5808b24 to f8621a3 Compare May 27, 2024 14:34
@flovogt flovogt force-pushed the manifest-transformer branch from f8621a3 to ba62e4d Compare July 9, 2024 14:47
flovogt added a commit to SAP/ui5-project that referenced this pull request Jul 12, 2024
@flovogt flovogt merged commit a39c8de into main Jul 12, 2024
@flovogt flovogt deleted the manifest-transformer branch July 12, 2024 13:59
d3xter666 pushed a commit to UI5/cli that referenced this pull request Sep 24, 2025
d3xter666 pushed a commit to UI5/cli that referenced this pull request Sep 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants