Skip to content

feat(configurator): import digit-configurator as configurator/ snapshot - #50

Closed
KDwevedi wants to merge 1 commit into
ChakshuGautam:nairobifrom
KDwevedi:feat/import-configurator-snapshot
Closed

feat(configurator): import digit-configurator as configurator/ snapshot#50
KDwevedi wants to merge 1 commit into
ChakshuGautam:nairobifrom
KDwevedi:feat/import-configurator-snapshot

Conversation

@KDwevedi

Copy link
Copy Markdown
Collaborator

Summary

  • Imports the configurator SPA from ChakshuGautam/digit-configurator as a top-level configurator/ subdirectory in CCRS.
  • Snapshot only — source files copied flat from main HEAD; no .git history, no node_modules, no build artifacts.
  • No integration changes. Nothing in backend/, local-setup/, ansible/, or nginx/ is touched. Build/serve wiring (e.g. an 09-ensure-configurator-style play, or a docker target) is intentionally deferred — to be designed in a follow-up.

Provenance

Source repo ChakshuGautam/digit-configurator
Source branch main
Source commit 068823f0
Files imported 302 (~2.7 MB)
Excluded .git/, node_modules/, dist/, build/, playwright-report/, test-results/, .cache/, .DS_Store

What's in configurator/

The full source tree: src/, packages/data-provider/, packages/digit-datagrid/, e2e/, e2e-onboarding/, public/, docs/plans/, plus the build config (vite.config.ts, tsconfig*.json, tailwind.config.js, postcss.config.js, eslint.config.js, vitest.config.ts), package.json + package-lock.json, index.html, and README.md.

What this PR does NOT do

  • No Dockerfile, no docker-compose service entry, no ansible task.
  • No nginx route. (Live deploys today serve the SPA from /var/www/configurator/; that path is unchanged.)
  • No CI hookup. (Lint/test/build for configurator/ will need its own workflow or a paths: filter on existing ones.)
  • No build-time vendoring of dist/. The intent is source-in-tree, build elsewhere — exact location TBD.

Re-syncing later

Since this is a flat snapshot rather than a subtree, future syncs are a rsync from a fresh digit-configurator checkout. If we want history preservation later, we can convert to git subtree in a follow-up — straightforward but a bigger diff.

Test plan

  • Confirm no .git, node_modules, or build artifacts under configurator/
  • Confirm existing CCRS build/test paths are unaffected (no shared package.json, no top-level config collisions)
  • Spot-check a handful of files match upstream 068823f0

🤖 Generated with Claude Code

Flat snapshot of ChakshuGautam/digit-configurator @ main (068823f0)
into a top-level configurator/ subdirectory. Source-only — no build
artifacts, no .git history, no node_modules. Integration (build,
serve, ansible wiring) is intentionally deferred.

Source: https://github.com/ChakshuGautam/digit-configurator
Commit: 068823f0415b5d94be998c2f299d8d25a1d19550
Files: 302 (~2.7 MB)
Excluded: .git/, node_modules/, dist/, build/, playwright-report/,
test-results/, .cache/, .DS_Store

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented May 20, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d4bbf5b1-8344-429c-9da9-e636c812dc75

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

KDwevedi added a commit to KDwevedi/Citizen-Complaint-Resolution-System that referenced this pull request May 20, 2026
…t in this PR)

Blank-dir Mac deploy from a wiped state surfaced: ~/digit/configurator/ is
empty after the playbook completes (no task in this PR builds the dist),
so /configurator/ returns 403 and the validate task fails.

digit-configurator is vendored separately in PR ChakshuGautam#50 (kept-separate per the
fold/supersede audit; 58k-line snapshot would drown ChakshuGautam#52's review surface).
Until ChakshuGautam#50 lands, operators have two paths:
  (a) leave configurator:false (default in the example now) — the rest of
      the stack still validates 200 across mdms/ui/status/auth
  (b) build digit-configurator dist externally, drop into ~/digit/configurator/,
      then flip nginx_features.configurator:true and re-deploy

MAC-QUICKSTART.md updated with the same guidance + an adjusted verify curl
sequence (4 probes instead of 5).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@KDwevedi

Copy link
Copy Markdown
Collaborator Author

Superseded by #52. Configurator is now built from source at deploy (build_configurator: truefiles/configurator-build.sh: clone digit-configurator + vite build) rather than vendored as a snapshot — same end (configurator served at /configurator/), but no committed snapshot to drift. Closing in favor of the build-from-source approach in #52.

@KDwevedi KDwevedi closed this May 21, 2026
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.

1 participant