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

Make robot go home when battery or pressure levels are outside safe limits #1684

Merged
merged 3 commits into from
Jul 25, 2024
Merged

Make robot go home when battery or pressure levels are outside safe limits #1684

merged 3 commits into from
Jul 25, 2024

Conversation

andchiind
Copy link
Contributor

@andchiind andchiind commented Jul 23, 2024 •
edited
Loading

Closes #1459

@andchiind andchiind added feature New feature or request backend Backend related functionality frontend Frontend related functionality labels Jul 23, 2024
@andchiind andchiind added this to the Ready for users milestone Jul 23, 2024
@andchiind andchiind self-assigned this Jul 23, 2024
@github-actions GitHub Actions
Copy link

🔔 Changes in database folder detected 🔔
Do these changes require adding new migrations? 🤔 In that case follow these steps.
If you are uncertain, ask a database admin on the team 😄

@andchiind andchiind force-pushed the 1469-abort-mission branch 4 times, most recently from d547e25 to 191e528 Compare July 24, 2024 10:07
@andchiind andchiind marked this pull request as ready for review July 24, 2024 10:08
@andchiind andchiind force-pushed the 1469-abort-mission branch 3 times, most recently from 7654e9c to 6160c55 Compare July 24, 2024 10:23
@github-actions GitHub Actions
Copy link

🔔 Migrations changes detected 🔔
📣 Remember to comment "/UpdateDatabase" after review approval for migrations to take effect!

@github-actions github-actions bot added the database-change Will require migration label Jul 24, 2024
@andchiind andchiind force-pushed the 1469-abort-mission branch 2 times, most recently from c3f1297 to 1cf204b Compare July 24, 2024 13:56
betaniat
betaniat reviewed Jul 25, 2024
View reviewed changes
Copy link
Contributor

@betaniat betaniat left a comment

Choose a reason for hiding this comment

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

LGTM :) I saw it working

@andchiind
Copy link
Contributor Author

/UpdateDatabase

@github-actions GitHub Actions
Copy link

👀 Running migration command... 👀

@github-actions GitHub Actions
Copy link

✨ Successfully ran migration command! ✨

@andchiind andchiind force-pushed the 1469-abort-mission branch from 1cf204b to 9e8a37c Compare July 25, 2024 12:54
@andchiind andchiind merged commit 652c2f0 into equinor:main Jul 25, 2024
12 checks passed
@andchiind andchiind deleted the 1469-abort-mission branch July 25, 2024 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@betaniat betaniat betaniat approved these changes

Assignees

@andchiind andchiind

Labels
backend Backend related functionality database-change Will require migration feature New feature or request frontend Frontend related functionality
Projects
None yet
Milestone
Ready for users
Development

Successfully merging this pull request may close these issues.

If robot battery is running low, stop the missions and send it home
2 participants
@andchiind @betaniat