Skip to content

Rename server-logics to server-logic in web extension#1526

Merged
amitjoshi438 merged 3 commits intomainfrom
users/amitjoshi/rename-serverlogic-folder
Mar 30, 2026
Merged

Rename server-logics to server-logic in web extension#1526
amitjoshi438 merged 3 commits intomainfrom
users/amitjoshi/rename-serverlogic-folder

Conversation

@amitjoshi438
Copy link
Copy Markdown
Contributor

@amitjoshi438 amitjoshi438 commented Mar 30, 2026

Summary

  • Renamed the server-logics folder display name to server-logic (removed trailing 's') in the VS Code web explorer
  • Only changed _foldername in portalSchema.ts (V1 and V2) and the SERVERLOGICS constant value in constants.ts
  • Entity names, enum values, and _vscodeentityname are unchanged — fetch pipeline is not affected
image

Test plan

  • Verify server-logic folder appears correctly in VS Code web explorer (no trailing 's')
  • Verify server logic files load successfully on first load
  • Verify server logic files can be saved back to Dataverse
  • Run npm run test-web-integration

🤖 Generated with Claude Code

Remove trailing 's' from the server-logics folder name and related
constants/enums to use singular form (server-logic) consistently.
Only VS Code-side naming is affected; Dataverse entity names remain unchanged.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@amitjoshi438 amitjoshi438 requested review from a team as code owners March 30, 2026 07:12
Amit Joshi and others added 2 commits March 30, 2026 14:25
Only change the folder display name (_foldername) and the corresponding
constant value. Entity names and enum values are unchanged to preserve
the fetch pipeline.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@amitjoshi438 amitjoshi438 merged commit 3c5d101 into main Mar 30, 2026
8 checks passed
@amitjoshi438 amitjoshi438 deleted the users/amitjoshi/rename-serverlogic-folder branch March 30, 2026 09:21
@amitjoshi438 amitjoshi438 mentioned this pull request Apr 9, 2026
2 tasks
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