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 Images #244

Merged
merged 10 commits into from
Feb 24, 2023

Conversation

szakhlypa
Copy link
Collaborator

Based on #240

Adds forms and logic to create, update and delete Base Images.

Create Base Image:

2023-02-10.09-20-28.mp4

Update and delete Base Image:

2023-02-10.09-23-20.mp4

@coveralls
Copy link

coveralls commented Feb 13, 2023

Pull Request Test Coverage Report for Build 4262194384

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

Totals Coverage Status
Change from base Build eab5551044719b95b3aef68c1dec7d916962d8d5: -80.9%
Covered Lines: 0
Relevant Lines: 0

💛 - Coveralls

frontend/src/components/BaseImagesTable.tsx Outdated Show resolved Hide resolved
frontend/src/pages/BaseImageCreate.tsx Outdated Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
npm run relay:generate-schema
npm run format

Signed-off-by: Sergey Zakhlypa <[email protected]>
- rename `BaseImage` type to `BaseImageInfo` to respect new schema
- add stubbed data to mock Relay's responses for BaseImage queries

Signed-off-by: Sergey Zakhlypa <[email protected]>
Add the BaseImagesTable React component

Signed-off-by: Sergey Zakhlypa <[email protected]>
Fetch BaseImageCollection's BaseImages and show them on
BaseImageCollection page

Signed-off-by: Sergey Zakhlypa <[email protected]>
npm install --save semver @types/semver

Signed-off-by: Sergey Zakhlypa <[email protected]>
Without React's batched updates re-render order is not guaranteed and
a child component can update before the parent. This way sometimes
react-relay hooks like `useFragment` can return null even for non-nullable
field(s).
Batching with `unstable_batchedUpdates` is a valid
[work around](facebook/react#16387 (comment)),
also starting from 18 version React uses [automatic batching](reactwg/react-18#21)
but continue to support `unstable_batchedUpdates`.

Signed-off-by: Sergey Zakhlypa <[email protected]>
Add a page with a form to create Base Image

Signed-off-by: Sergey Zakhlypa <[email protected]>
Add a page with a form to view and edit Base Image

Signed-off-by: Sergey Zakhlypa <[email protected]>
npm run i18n:extract

Signed-off-by: Sergey Zakhlypa <[email protected]>
- update base images changes
- add periods at the end of changes where they're omitted to fix [inconsistency](olivierlacan/keep-a-changelog#450)

Signed-off-by: Sergey Zakhlypa <[email protected]>
@szakhlypa szakhlypa marked this pull request as ready for review February 24, 2023 12:03
@rbino rbino merged commit b09c212 into edgehog-device-manager:main Feb 24, 2023
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.

4 participants