Skip to content

Related displays without files#223

Merged
adedamola-sode merged 10 commits into
mainfrom
related-displays-without-files
May 21, 2026
Merged

Related displays without files#223
adedamola-sode merged 10 commits into
mainfrom
related-displays-without-files

Conversation

@adedamola-sode
Copy link
Copy Markdown
Collaborator

Changes include:

  1. Modifying models to hold either a file, or a remote_screen
  2. Changing the allocate widget function in generate.py to try for a file first, if not, select the remote_screen key instead as it is either one or the other
  3. Passing the IOC macro to all generated action buttons

@codecov
Copy link
Copy Markdown

codecov Bot commented May 20, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.31%. Comparing base (2f86a73) to head (2b1add1).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #223      +/-   ##
==========================================
+ Coverage   94.77%   95.31%   +0.54%     
==========================================
  Files          10       10              
  Lines         861      854       -7     
==========================================
- Hits          816      814       -2     
+ Misses         45       40       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@adedamola-sode
Copy link
Copy Markdown
Collaborator Author

adedamola-sode commented May 21, 2026

This should have been a separate commit, but the testing was done together:

  1. Added an if branch for embedded screens in json map, to add the children of the embedded screen to the current node skipping the embedded screen itself
  2. Modified the tests for json_map_embedded_screens

@adedamola-sode adedamola-sode merged commit 49cabfa into main May 21, 2026
8 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