Skip to content

fix: lowercase urlName in generated sfdc_cms__site content @W-21736591@ - #770

Merged
mshanemc merged 1 commit into
mainfrom
fix/lowercase-site-urlname
Mar 26, 2026
Merged

fix: lowercase urlName in generated sfdc_cms__site content @W-21736591@#770
mshanemc merged 1 commit into
mainfrom
fix/lowercase-site-urlname

Conversation

@k-j-kim

@k-j-kim k-j-kim commented Mar 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • enforce lowercase urlName values for generated sfdc_cms__site/*/content.json files after template name replacement
  • keep scope narrow by applying the normalization only to content.json under sfdc_cms__site
  • add focused unit tests covering lowercase conversion and no-op behavior for unrelated files and invalid JSON

Test plan

  • npx mocha --require ts-node/register 'test/utils/webappTemplateUtils.test.ts'
  • npx mocha --require ts-node/register 'test/**/*.test.ts'
  • git push pre-push hooks (yarn build, yarn test) pass

@W-21736591@

Normalize sfdc_cms__site content.json urlName values to lowercase.
This keeps generated Digital Experience site URLs compliant.

Made-with: Cursor
@k-j-kim
k-j-kim requested a review from a team as a code owner March 26, 2026 05:46
@k-j-kim
k-j-kim requested a review from RitamAgrawal March 26, 2026 05:46
@k-j-kim k-j-kim changed the title fix: lowercase urlName in generated sfdc_cms__site content fix: lowercase urlName in generated sfdc_cms__site content @W-21736591@ Mar 26, 2026
@mshanemc
mshanemc merged commit 74bbe6e into main Mar 26, 2026
15 of 18 checks passed
@mshanemc
mshanemc deleted the fix/lowercase-site-urlname branch March 26, 2026 18:53
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.

3 participants