Restructure samples into category-first taxonomy (apps / snippets / traditional / server-logic)#109
Merged
Conversation
Reorganize the samples tree so full reference applications and single-topic snippets are clearly separated, and non-code sites + server logic live under a new traditional/ category: - spa/apps/<name>/<framework>/ full multi-feature apps (car-sales react+angular, credit-cards react, vue-admin vue) - spa/snippets/<topic>/ single-topic React samples (environment-variables, localization, authentication, fluent-ui, cloud-flow, file-upload/*) - traditional/server-logic/ server-side logic samples (sharepoint-integration) All moves preserve history (git rename detection). Root README.md and samples/spa/README.md index tables regrouped into Apps/Snippets with corrected paths; a new Traditional category is documented. Fixed stale cd/link paths and self-referential folder names in the affected sample READMEs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…apply review follow-ups - Move server-logic to samples/server-logic/ (peer of spa/ and traditional/) - Add traditional/edm-liquid-webtemplate-starter/ placeholder for first EDM sample - Fix app README deploy paths (cd into <name>/<framework>) - Add apps-vs-snippets classification + graduation rule to CONTRIBUTING.md - Cross-link the two SharePoint samples; add traditional/ and server-logic/ index READMEs - Update root README to three top-level categories Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…/P2/P3) - Add MIGRATION.md (old->new moved-paths table) + link from README (P1) - Document snippet variant-family layout spa/snippets/<topic>/<variant>/ (P1) - Add server-logic -> extensions/ graduation trigger to CONTRIBUTING (P2) - Tighten apps-vs-snippets rule with objective signals + <topic>-><name> naming (P2/P3) - Point EDM placeholder at a tracking issue for ownership (P3) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
priyanshu92
approved these changes
Jul 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What & why
Reorganizes
samples/from a framework-first layout (spa/react/,spa/angular/,spa/vue/) to a category-first taxonomy, to prepare the repo for a large influx of single-topic samples. No source code changed — the 14 existing samples were relocated viagit mv(history preserved), with docs reconciled.New structure
Changes
spa/apps/,spa/snippets/,traditional/,server-logic/(3937 pure renames, byte-identical).README.md,samples/spa/README.md, and new index READMEs fortraditional/andserver-logic/; allcd/link/deep-link paths fixed; the two SharePoint samples now cross-link.CONTRIBUTING.md: documents the layout, an apps-vs-snippets classification rule with objective signals, the snippet-variant-family pattern, and graduation triggers (snippet -> app, and server-logic -> a futureextensions/parent).MIGRATION.md: old -> new moved-paths table (GitHub does not redirect renamed paths), linked from the README intro.traditional/, clearly marked non-functional.Validation
npm install) succeed; builds are gated only by the local Node version floor (environmental, not path-related).Out of scope (pre-existing, not touched)
server-logic/sharepoint-integration/readme.md..powerpages-sitebundle snapshots incloud-flow,file-upload/notes,vue-admin-template.Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com