Skip to content
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

[frontend] Add Base Image Collections #230

Merged

Conversation

szakhlypa
Copy link
Contributor

@szakhlypa szakhlypa commented Dec 6, 2022

Adds forms and logic to create, update and delete Base Image Collections.

System model is required to create a new Base Image Collection.
bic_no_system_model

Create, filter and delete Base Image Collection

bic_create_filter_delete.mp4

Update Base Image Collection

bic_update.mp4

@coveralls
Copy link

coveralls commented Dec 6, 2022

Pull Request Test Coverage Report for Build 3891120810

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-80.7%) to 0.0%

Totals Coverage Status
Change from base Build 37886bc6e1d006629b943c7bc4fab65a76c63713: -80.7%
Covered Lines: 0
Relevant Lines: 0

💛 - Coveralls

@coveralls
Copy link

Pull Request Test Coverage Report for Build b77c677eb04b64e35e9fb6cef9e5ad22ded21018-PR-230

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+81.005%) to 81.005%

Totals Coverage Status
Change from base Build 3619909826: 81.005%
Covered Lines: 887
Relevant Lines: 1095

💛 - Coveralls

@szakhlypa szakhlypa force-pushed the frontend-base-image-collections branch from b77c677 to da2ec49 Compare January 11, 2023 08:45
@szakhlypa szakhlypa marked this pull request as ready for review January 11, 2023 12:15
frontend/src/pages/BaseImageCollections.tsx Outdated Show resolved Hide resolved
frontend/src/forms/UpdateBaseImageCollection.tsx Outdated Show resolved Hide resolved
frontend/src/pages/BaseImageCollection.tsx Outdated Show resolved Hide resolved
npm run relay:generate-schema
npm run format

Signed-off-by: Sergey Zakhlypa <sergey.zakhlypa@secomind.com>
Add the BaseImageCollectionsTable React component

Signed-off-by: Sergey Zakhlypa <sergey.zakhlypa@secomind.com>
Add stubbed data to mock Relay's responses for BaseImageCollection queries

Signed-off-by: Sergey Zakhlypa <sergey.zakhlypa@secomind.com>
Add a page to fetch and list available Base Image Collections

Signed-off-by: Sergey Zakhlypa <sergey.zakhlypa@secomind.com>
@szakhlypa szakhlypa force-pushed the frontend-base-image-collections branch from da2ec49 to a0dee1a Compare January 17, 2023 08:40
Add a page with a form to view and edit Base Image Collection

Signed-off-by: Sergey Zakhlypa <sergey.zakhlypa@secomind.com>
@szakhlypa szakhlypa force-pushed the frontend-base-image-collections branch from a0dee1a to 6d61462 Compare January 18, 2023 09:55
Add a page with a form to create Base Image Collection

Signed-off-by: Sergey Zakhlypa <sergey.zakhlypa@secomind.com>
npm run i18n:extract

Signed-off-by: Sergey Zakhlypa <sergey.zakhlypa@secomind.com>
Signed-off-by: Sergey Zakhlypa <sergey.zakhlypa@secomind.com>
@rbino rbino merged commit aae3f98 into edgehog-device-manager:main Jan 19, 2023
@coveralls
Copy link

coveralls commented Nov 15, 2024

Pull Request Test Coverage Report for Build 3627995462

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 0.0%

Totals Coverage Status
Change from base Build 3619909826: 0.0%
Covered Lines: 0
Relevant Lines: 0

💛 - Coveralls

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.

5 participants