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

Project board fixes #96

Merged
merged 72 commits into from
Sep 5, 2022
Merged

Project board fixes #96

merged 72 commits into from
Sep 5, 2022

Conversation

maggie44
Copy link
Contributor

@maggie44 maggie44 commented Aug 16, 2022

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[X] Bug fix
[X] New feature
[ ] Other, please explain:

What changes did you make? (Give an overview)
Various fixes from the project board.

Which issue (if any) does this pull request address?

Closes: #79
Closes: #69

Is there anything you'd like reviewers to focus on?

For now development should be done by pushing the code to a device on your local network, then starting the UI with the following command on your local system:

HOSTNAME=balena.local yarn workspace ui dev

The UI will then start locally with hot reload but draw on the running device on your network for the backend, allowing functionality like env setting.
@maggie44 maggie44 marked this pull request as draft August 16, 2022 10:00
@maggie44 maggie44 marked this pull request as ready for review August 17, 2022 10:49
@maggie44
Copy link
Contributor Author

maggie44 commented Aug 17, 2022

@nucleardreamer I have run out of days this week to finish this off but will pick it up again next week. A lot of ticks in the box already though.

If you want you can start looking over the changes so far and I will continue next week in a new PR.

@maggie44 maggie44 removed the request for review from nucleardreamer August 22, 2022 12:24
@maggie44
Copy link
Contributor Author

maggie44 commented Aug 31, 2022

@nucleardreamer this should do it for now. Looking to merge this one. Not sure how to go about the review, maybe just have a scan over for anything obviously problematic, or anything that doesn't build or display properly. Will do a proper bug discovery the dog food way. Once it's merged Weblate will update itself and hopefully we will get the string translations contributed for the new and changed items.

@nucleardreamer
Copy link
Contributor

I now have time to review this, so I figure I will run it on a couple devices and make sure there isn't anything outstanding? @Maggie0002

@maggie44
Copy link
Contributor Author

maggie44 commented Sep 1, 2022

I now have time to review this, so I figure I will run it on a couple devices and make sure there isn't anything outstanding? @Maggie0002

I think that make sense. After this PR the next one will make things easier, because the next one will build all the apps too and will be available as assets in the PR.

For now, there is the normal interface as usual, and if you go to yourhostname.local/app you will find the PWA version to play around with too. I haven't added any direct link to it from the dashboard, it's an experimental thing to identify how best to utilise it (in light of the TLS issues I mentioned yesterday).

@nucleardreamer
Copy link
Contributor

Alright! I didn't run into any visible problems myself with this whole PR. I think its fine to roll without linking the PWA for now like you mentioned. This is fantastic!!

@maggie44 maggie44 merged commit 7352d25 into main Sep 5, 2022
@maggie44 maggie44 deleted the dev-updates branch September 9, 2022 09:11
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.

Automate block publishing to balenaHub Make device info mobile friendly
2 participants