Conversation
…y and consistency
…nality in summary screen
…es and update summary screen functionality
- Allow multiple instances of the same device to be added to a room. - Introduce DeviceResultScreen to display results for each device instance. - Implement risk assessment features for devices and rooms, including risk levels and recommendations. - Update UI components to reflect device instance counts, completion status, and risk levels. - Add search functionality in DeviceSelectionScreen for easier device management. - Ensure proper navigation flow between device questionnaire and results screens. - Update tests to cover new functionalities and ensure reliability.
- Updated CatalogData to replace string device types with DeviceCategory enum values for better type safety and maintainability. - Modified DeviceTemplate and DeviceInstance classes to accommodate the new DeviceCategory enum. - Adjusted various screens and dialogs to utilize the updated device type handling. - Introduced l10n_extensions.dart for easier localization access in the widget tree. - Enhanced the WelcomeScreen and other screens to pass the SurveyState instance correctly. - Updated widget tests to reflect changes in the app structure and ensure proper initialization.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This pull request updates the Simplications project to switch Windows release packaging from MSIX to a classic Inno Setup installer, and removes the
.instructions.mdfile. It also improves and consolidates documentation for development, release, and CI/CD processes, and refines device type handling in the.llmrccontext file. Below are the most important changes:Windows Release Packaging and CI/CD:
.github/workflows/release.yml. [1] [2] [3].github/installer/simplications.issto define the Windows installer build process.Documentation and Development Process:
.instructions.md(which contained development, localization, and versioning guidelines), consolidating and updating its content intoAGENTS.mdand.llmrc.AGENTS.mdwith explicit release tagging instructions, more detail on supported platforms, localization workflow, PDF export character requirements, and code style/testing guidelines. [1] [2] [3] [4]Device Type Handling and Context:
.llmrcto clarify that device types are now represented by theDeviceCategoryenum (not strings), updating all related instructions, code references, and device-specific question logic. [1] [2] [3] [4] [5] [6]These changes streamline the release process, improve maintainability, and ensure documentation accurately reflects the current codebase and workflow.
Type of change
Checklist
flutter analyzepasses with no new issuesflutter testpasses