Skip to content

Appsmith UI updates for bare entity IDs - #124

Open
Oluwajuwon-O wants to merge 9 commits into
datakind:mainfrom
Oluwajuwon-O:feature/appsmith_ui_updates
Open

Appsmith UI updates for bare entity IDs#124
Oluwajuwon-O wants to merge 9 commits into
datakind:mainfrom
Oluwajuwon-O:feature/appsmith_ui_updates

Conversation

@Oluwajuwon-O

@Oluwajuwon-O Oluwajuwon-O commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR addresses the Appsmith UI configuration issues described in the technical documentation:

Unreadable UI Configurations and Fragile Prefix Stripping

  • Previously, the exported Appsmith JSON payload was minified, making it impossible to review UI changes in Git.
  • The UI was hardcoded to use legacy, healthcare-specific test parameters and relied on fragile .replace() string manipulation inside JSFunctions.getEntityColumns to strip dot_model__ prefixes from entity IDs.

Resolution

  • JSON Reformatting: Structurally un-minified and pretty-printed DOT App V2.json. While this generates a massive line-count diff, it introduces no behavioral changes by itself and makes the file readable for future PR reviews.
  • Domain-Neutral Parameters: Updated the possible_duplicate_forms test form fields to use the new generic keys (date_column, group_column, id_column, period) to align with the backend dbt macro updates.
  • Bare Entity IDs: Reconfigured dropdown menus and JSON generating blocks to actively pass and save bare entity IDs. Eliminated the fragile .replace() prefix stripping from JSFunctions.getEntityColumns.

Asana Task

Deployment Readiness*

Testing

Describe or check:

  • Created or updated unit, feature, and/or integration tests
  • Typical manual testing in the local env browser, dev pipeline, etc.

Deployment Notes

Describe or check:

  • No special deployment steps required

Rollback Plan

Describe or check:

  • Standard revert is sufficient (git revert)

Reviewer Guidance / Questions*

Screenshots / Testing Evidence*

SOC 2 Change Management Checklist

  • None of the below are true in this code
  • New roles/permissions are introduced without review and approval by the product manager
  • Hardcoded credentials, secrets, or API keys are present in this code
  • Secrets are being managed outside of the approved secrets management process (e.g., GitHub Secrets, environment variables)
  • PII or sensitive data handling is introduced or changed without being reviewed against our data classification policy
  • Sensitive data is written to logs
  • Input validation and sanitization is missing
  • An unnecessary attack surface has been introduced (e.g., unused endpoints, open ports, debug modes left enabled)
  • Common vulnerabilities have been introduced in the code (inc. any dependencies added or updated)
  • No review for common vulnerabilities has been conducted
  • Not tested in a non-production environment
  • Breaking changes to existing APIs or integrations with downstream consumers being notified
  • Performance impact has not been considered or acceptable
  • Appropriate audit logging is missing for any security-relevant actions introduced by this change
  • Log entries contain sensitive or PII data
  • All existing tests do not pass locally (./vendor/bin/pest)

Provide justification if you are submitting a PR with any boxes checked other than the first.


Reminder for Reviewers: By approving this PR you are confirming that you have reviewed the code for correctness, security, and compliance with our engineering and SOC 2 standards. Do not approve PRs where SOC 2 checklist items are checked without documented justification.

*Optional


Align DOT App V2 test forms with bare entity references and the
generic possible_duplicate_forms keys (date_column, group_column,
id_column, period).
Same CI pylint threshold issue as datakind#122; this stacked branch still carried
the pre-fix test module and failed lint at 6.79 (< 7.0).
Integration CI failed because the expected UUID was computed from
alphabetical JSON key order, while the insert trigger's jsonb cast
produces a different deterministic test_id.
Postgres jsonb returns keys in a deterministic order that differs from
the hand-edited fixture string; align the expected summary row.
…e/appsmith_ui_updates

Bring forward parent CI fixes: pylint disable, fixture test_id, and
jsonb test_parameters key order.
@Oluwajuwon-O
Oluwajuwon-O marked this pull request as ready for review August 7, 2026 10:01
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