Skip to content

chore: update documentation and CI linting - #825

Merged
yeasy merged 2 commits into
hyperledger-cello:mainfrom
Pranav-d33:chore/docs-and-ci
Aug 10, 2026
Merged

chore: update documentation and CI linting#825
yeasy merged 2 commits into
hyperledger-cello:mainfrom
Pranav-d33:chore/docs-and-ci

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:25
Signed-off-by: Pranav dhiran <dhiranpranav72@gmail.com>
@Pranav-d33

Copy link
Copy Markdown
Contributor Author

#818

@yeasy
yeasy merged commit 97dfe73 into hyperledger-cello:main Aug 10, 2026
3 checks passed
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