Skip to content

Comments

Feature/row builder widget#226

Merged
darinkelkhoff merged 5 commits intodevelopfrom
feature/row-builder-widget
Sep 19, 2025
Merged

Feature/row builder widget#226
darinkelkhoff merged 5 commits intodevelopfrom
feature/row-builder-widget

Conversation

@darinkelkhoff
Copy link
Contributor

Pull Request

📝 Description

What does this PR do?
Adds new RowBuilder Widget to QQQ.

This widget has a much bigger footprint on the material-dashboard side. Here in qqq, there's just a new enum entry, a new base Render class, with a meta-data Builder.

There's also a fix for an intermittent swapping bug i'd see sometimes locally.

Related Issue:
Closes #[issue_number]

🔍 Type of Change

  • Bug Fix - Fixes an existing issue
  • New Feature - Adds new functionality
  • Breaking Change - Fix or feature that would cause existing functionality to not work as expected
  • Documentation Update - Updates documentation
  • Refactoring - Code change that neither fixes a bug nor adds a feature
  • Performance Improvement - Code change that improves performance
  • Test Addition - Adding missing tests or correcting existing tests

🧪 Testing

How has this been tested?

  • Unit Tests: All unit tests pass
  • Integration Tests: Integration tests pass
  • Coverage: Meets QQQ's coverage requirements (80% instructions, 95% classes)
  • Manual Testing: Tested manually in development environment

Test Commands:

# Commands used for testing
mvn test
mvn verify

📋 Checklist

Before submitting this PR, please ensure:

  • Code Style: Follows QQQ's Code Review Standards
  • Tests: All tests pass with required coverage
  • Documentation: Updated relevant wiki pages and code comments
  • Breaking Changes: Documented any breaking changes
  • Commit Messages: Follow conventional commit format
  • Self Review: Code has been reviewed by the author

🔗 Related Resources

Documentation:

📊 Additional Information

Any additional information that reviewers should know:

  • Screenshots (if applicable)
  • Performance impact
  • Migration notes
  • Dependencies added/removed

Thank you for contributing to QQQ! 🚀

Complete and usable for standalone-widget use case (e.g., in a workflow) or for a process.

Not quite ready for use in record view/edit screens.
…unctionality as new public validateField method.

Note that this changes the table passed in to fieldBehavior.validateBehaviorConfiguration to now be possibly null (before it never would have been).

Thus update all implementors of validateBehaviorConfiguration to handle null table.
…of the QInstance.

This fixes some situations where multiple threads could be hot swapping simultaneously, and some static props (e.g., MetaDataProducerHelper.tableMetaDataCustomizer) could get messy.  Could/should fix that there too, but, this change makes sense on its own terms as well.
@github-actions
Copy link

🔒 Security Scan Complete

Dependency vulnerability scan completed for this PR. View full report

@darinkelkhoff darinkelkhoff merged commit 90cf70d into develop Sep 19, 2025
12 of 17 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