Skip to content

Conversation

@sofyakurilova
Copy link
Collaborator

Merge dev_ui with dev after release

@sofyakurilova sofyakurilova requested a review from jboddey February 7, 2024 14:24
@sofyakurilova sofyakurilova self-assigned this Feb 7, 2024
@jboddey jboddey self-requested a review February 7, 2024 17:21
@jboddey
Copy link
Contributor

jboddey commented Feb 7, 2024

@sofyakurilova I have found a bug in this PR:
When the user saves the connection settings, the payload sent to the API is as follows:
{ "network": { device_intf: "0", internet_intf: "1" }}

This results in invalid interfaces being saved in the system config. The UI should still return the actual interface name to the API.

@jboddey jboddey added the ui Bug or request that impacts the user interface label Feb 7, 2024
sofyakurilova and others added 8 commits February 8, 2024 10:30
- feat store
- feat start test run from devices page
* Fix styles for helperbird

* Fix node version
Adds ESLint workflow
* Adds store for system config

* Adds error message callout if selected port is missing
…t is saved (#253)

Co-authored-by: Volha Mardvilka <mardvilka@google.com>
@jboddey
Copy link
Contributor

jboddey commented Feb 8, 2024

The bug is now resolved. @KrystsinaT could you verify perhaps?

@jboddey jboddey requested a review from KrystsinaT February 8, 2024 10:38
@sofyakurilova sofyakurilova merged commit 6f17edb into dev Feb 8, 2024
@jboddey jboddey deleted the feature/dev_ui branch February 15, 2024 17:23
hitnik pushed a commit that referenced this pull request Jul 22, 2024
* Update UI
- feat store
- feat start test run from devices page

* Fix styles for helperbird (#243)

* Fix styles for helperbird

* Fix node version

* Adds ESLint workflow (#248)

Adds ESLint workflow

* Feature/317232143 (#247)

* Adds store for system config

* Adds error message callout if selected port is missing

* Set snack bar duration to 0 to prevent auto closing (#250)

* Adds trap focus on connection settings panel (#251)

* Disables settings save button if there is no interfaces (#252)

* 323349282: (fix) [ui] change logic to hide message if only device port is saved (#253)

Co-authored-by: Volha Mardvilka <mardvilka@google.com>

---------

Co-authored-by: Olga Mardvilko <omardvilko@mail.ru>
Co-authored-by: Volha Mardvilka <mardvilka@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui Bug or request that impacts the user interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants