Skip to content

fix: registration workflow and docker compose setup - #828

Open
Pranav-d33 wants to merge 2 commits into
hyperledger-cello:mainfrom
Pranav-d33:fix/registration-and-compose
Open

fix: registration workflow and docker compose setup#828
Pranav-d33 wants to merge 2 commits into
hyperledger-cello:mainfrom
Pranav-d33:fix/registration-and-compose

Conversation

@Pranav-d33

@Pranav-d33 Pranav-d33 commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

This PR addresses local development environment setups and resolves blockages in the user registration workflow. It introduces necessary fixes to allow development stacks to run end-to-end without being blocked by misconfigured database credentials or mock-only configurations.

Changes

  • Docker Compose: Switched dashboard image configuration in docker-compose.dev.yml to a local developer build target and defaulted the DASHBOARD_SERVICE_PORT to 8081 to handle empty environment variables gracefully.
  • Login Component & Registration Page: Patched form inputs and the state management mappings in src/dashboard/src/components/Login/ and src/dashboard/src/pages/User/Login.js to correctly propagate values to registration endpoints.
  • Mock Data: Disabled default dashboard mocks in src/dashboard/config/config.js to ensure the UI communicates with the actual live django backend service during integration tests.
  • Package & Dependency Locks: Updated root and dashboard locks to ensure package compatibility.

Verification

  • Tested locally by booting the dev environment via make commands.
  • Verified that registering an organization no longer fails due to missing field mappings or backend communications.

@Pranav-d33
Pranav-d33 requested a review from a team as a code owner August 1, 2026 16:26
Signed-off-by: Pranav dhiran <dhiranpranav72@gmail.com>
@Pranav-d33
Pranav-d33 force-pushed the fix/registration-and-compose branch from 8159f32 to 35b2c56 Compare August 1, 2026 17:11
@Pranav-d33

Copy link
Copy Markdown
Contributor Author

#818

@yeasy
yeasy requested a review from YoungHypo August 10, 2026 04:24
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.

2 participants