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.

Sign up for GitHub

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

Jump to bottom

Add wizard #1589

Merged
merged 8 commits into from
Apr 9, 2023
Merged

Add wizard #1589

merged 8 commits into from
Apr 9, 2023

Conversation

patrickelectric
Copy link
Member

@patrickelectric patrickelectric commented Apr 8, 2023 •
edited

Needs #1582 and #1585
Peek 08-04-2023 16-01

Add wizard setup that opens on the first time that BlueOS shows
Sets vehicle name and mdns name to be used

@patrickelectric patrickelectric force-pushed the add-wizard branch 9 times, most recently from 1f16c3d to 40c40ef Compare April 8, 2023 22:20
@patrickelectric patrickelectric marked this pull request as ready for review April 8, 2023 22:21
Williangalvani
Williangalvani approved these changes Apr 9, 2023
View reviewed changes
Copy link
Member

@Williangalvani Williangalvani 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, I guess we can get rid of that console.log though

core/frontend/src/components/wizard/Wizard.vue Outdated Show resolved Hide resolved
Williangalvani and others added 8 commits April 8, 2023 22:14
@Williangalvani @patrickelectric
core: beacon: reduce logging
e0095ca
@Williangalvani @patrickelectric
core: beacon: add endpoint to change mdns hostname
c5bbc79
@Williangalvani @patrickelectric
core: beacon: remove unnecessary manager.save()
578ac98
@Williangalvani @patrickelectric
core: beacon: properly reload settings
aa0e1f2
@Williangalvani @patrickelectric
core: beacon: add endpoints to set vehicle name
181ff4b 
Co-authored-by: Willian Galvani <williangalvani@gmail.com>
Co-authored-by: Patrick José Pereira <patrickelectric@gmail.com>
@patrickelectric
core: frontend: components: Wizard: Add first version
40c4523 
Signed-off-by: Patrick José Pereira <patrickelectric@gmail.com>
@patrickelectric
core: frontend: App: Add wizard
e3a323d 
Signed-off-by: Patrick José Pereira <patrickelectric@gmail.com>
@patrickelectric
core: frontend: store: beacon: Add support to hostname and vehicle_name
9eb5582 
Signed-off-by: Patrick José Pereira <patrickelectric@gmail.com>
@patrickelectric patrickelectric merged commit 872685f into bluerobotics:master Apr 9, 2023
4 checks passed
@ES-Alexander ES-Alexander added the docs-needed Change needs to be documented label Jun 7, 2023
@ES-Alexander ES-Alexander mentioned this pull request Jun 19, 2023
Merged
16 tasks
@ES-Alexander ES-Alexander added docs-complete Change documentation has been completed and removed docs-needed Change needs to be documented labels Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Williangalvani Williangalvani Williangalvani approved these changes

Assignees
No one assigned
Labels
docs-complete Change documentation has been completed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@patrickelectric @Williangalvani @ES-Alexander