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

feat: user debug page dynamic updates #2839

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chrismclarke
Copy link
Member

@chrismclarke chrismclarke commented Mar 10, 2025

PR Checklist

  • PR title descriptive (can be used in release notes)

Description

Update user debug page to:

  • Keep dynamic data datable live-updated with data changes
  • Display full row data with visual distinction between default and overwritten values

Review Notes

Open a template that users dynamic data with user debug page sidebar, e.g.

http://localhost:4200/template/feat_data_actions(sidebar:user)

Author Notes

The dynamic data table will only show the list of tables where data has been set, hence at the start of the demo video only a single table is available.

When updating values the user debug table should reflect the updated values, showing in darker text the values that have been pulled from dynamic data vs default values. In cases where the dynamic data is updated to the same value as the default data, the text still appears darker as the update still exists in the database (would be quite a bit of added effort to always cross-check updated data against original, and more simple to just track all state changes instead)

Dev Notes

The debug page code has been refactored to make more use of signals to handle live data updating. Given the amount of code related just to the dynamic data table it might be better split into a separate component, but kept as-is for now in case useful to see the git diffs during review

I decided to add this PR independent to the work on the authored table component as that would require quite a bit of additional follow-up to accept the input data rows from the dynamic data subscription as well as custom row styling

Git Issues

Closes #2825
Closes #2826

Screenshots/Videos

Example - feat_data_actions template with user sidebar updating

Screenity.video.-.Mar.10.2025.mp4

@github-actions github-actions bot added the feature Work on app features/modules label Mar 10, 2025
@chrismclarke chrismclarke marked this pull request as ready for review March 10, 2025 20:39
@github-actions github-actions bot added feature Work on app features/modules and removed feature Work on app features/modules labels Mar 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Work on app features/modules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

user sidebar should show live updates Show default values in user tables
1 participant