Skip to content

Conversation

@jboddey
Copy link
Contributor

@jboddey jboddey commented Feb 12, 2025

No description provided.

MariusBaldovin and others added 30 commits August 29, 2024 13:35
* added tests for report endpoints

* added tests for certificate endpoints

* fix report.json

* removed responses library, not used in testing

* updated the test for /system/version endpoint

* fix pylint error

* improved the code structure

* removed timestamp variable in few tests

* updated create_report_folder and test_system_version

* fixed test_api.py failing tests

* removed 2 unnecessary lines

* fixed f string error

* corrected the testrun session variable name

---------

Signed-off-by: Marius <86727846+MariusBaldovin@users.noreply.github.com>
Co-authored-by: MariusBalovin <marius.baldovin@forestrock.co.uk>
Co-authored-by: J Boddey <boddey@google.com>
* Add additional device properties to config

* Fix pylint

* Fix types
…t present (#731)

* Prevent starting if device is not configured

* Fix pylint
* Remove default values from config

* Remove further reference
* Adds dialog if profile is successfully saved
* Open Edit device window when delete device is cancelled
* updated delete device endpoint to respond 404 if report is not found

* fix test_delete_report_no_report

* fixed failing tests

* fixed json formatting

* fixed device json
* jinja2 dependencies

* risk report styles file

* generating resk profile report using jinja

* remove unused code

* rename templates

* jinja2 test report

* change Device configuration  margin

* pylint
… by Orca screen reader (#760)

* 365059335: (fix) [GAR 1.1] change device tile structure to allow read by Orca screen reader

* 365059335: (fix) [GAR 1.1] change device tile - remove unnecessary aria label
Copy link
Collaborator

@OlgaMardvilko OlgaMardvilko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Lets wait for final testing.

@jboddey jboddey marked this pull request as ready for review February 12, 2025 16:29
@jboddey jboddey merged commit faa6cd3 into main Feb 20, 2025
20 checks passed
@jboddey jboddey deleted the release/v2.1.1 branch February 20, 2025 20:38
@jboddey jboddey restored the release/v2.1.1 branch February 20, 2025 20:42
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.

6 participants