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
Device release page #60
Conversation
lib/firmware_body_page.dart
Outdated
State<FirmwareBodyPage> createState() => _FirmwareBodyPageState(); | ||
} | ||
|
||
class _FirmwareBodyPageState extends State<FirmwareBodyPage> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FirmwareBodyPage
could be a StatelessWidget
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, yes of course.. that was kind of the point 😆
test/device_model_test.dart
Outdated
}); | ||
|
||
test('verify', () async { | ||
final device = testDevice(id: ''); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here, could you set a non-empty id please :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will do
test/device_model_test.dart
Outdated
}); | ||
|
||
test('install release', () async { | ||
final device = testDevice(id: ''); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
would be nice to specify a non-empty ID so it wouldn't match any default-constructed device below
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will do
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Show the release dialog as a page in the new layout. Design updates will follow.
DeviceBodyPage
- renders the right pane in the master detail layout for each deviceDeviceModel
- view model forDeviceBodyPage
install()
,verify()
for the device by calling the respective methods of theFirmwareModel
DeviceBodyPage
will respond accordingly