Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Service delivery reporting features #60

Open
wants to merge 55 commits into
base: main
Choose a base branch
from

Conversation

qhouyee
Copy link
Contributor

@qhouyee qhouyee commented Jan 28, 2025

This pull request adds the following features to support an initial pricing model implementation

  • Added form branches and arrays
  • Added option to generate report
  • Updated form extraction workflow following changes in the backend
  • Consolidation of form parsing methods
  • Improve dark mode design for the registry

@qhouyee qhouyee self-assigned this Jan 28, 2025
@qhouyee qhouyee linked an issue Jan 28, 2025 that may be closed by this pull request
2 tasks
…w row automatically when any empty form inputs in the last row array is filled
…he last row in the array to do nothing as an empty row
…directly from the SHACL in restriction and parse the list of classes
@qhouyee qhouyee force-pushed the qhouyee/29/service-delivery-reporting branch from 9b694e2 to c44c7ca Compare February 21, 2025 12:08
…tion that renders different form fields based on the selected dropdown
… also ingest form array and unregister previous branches
qhouyee and others added 21 commits February 24, 2025 16:50
…arents but parent object must not be displayed
…property field name rather than field id for server action endpoint
@qhouyee qhouyee marked this pull request as ready for review March 7, 2025 11:21
@qhouyee qhouyee requested a review from Ushcode March 7, 2025 11:21
Copy link
Member

@Ushcode Ushcode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

have not tested with working backend, but changes to code appear not to have any breaking logic. Happy to merge once tested with vis-backend-agent

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.

Service Delivery Reporting
2 participants