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

Remove Inspection Target #1400

Merged
merged 4 commits into from
Feb 9, 2024

Conversation

UsamaEquinorAFK
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Feb 8, 2024

🔔 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 😄

@UsamaEquinorAFK UsamaEquinorAFK changed the title Inspection type lovemachine Remove Inspection type Feb 8, 2024
@UsamaEquinorAFK UsamaEquinorAFK force-pushed the InspectionTypeLovemachine branch 2 times, most recently from d580f28 to e1ea1cd Compare February 8, 2024 09:23
Copy link

github-actions bot commented Feb 8, 2024

🔔 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 Feb 8, 2024
@UsamaEquinorAFK UsamaEquinorAFK added the bug Something isn't working label Feb 8, 2024
@UsamaEquinorAFK UsamaEquinorAFK changed the title Remove Inspection type Remove Inspection Target Feb 8, 2024
Copy link
Contributor

@prasm313 prasm313 left a comment

Choose a reason for hiding this comment

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

LGTM, tested without any bugs. Suggest that @Afonso-2403 takes a look before it gets approved.

Copy link
Contributor

@Afonso-2403 Afonso-2403 left a comment

Choose a reason for hiding this comment

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

LGTM, let's not merge yet until we have the workflow in place to update the existing missions

@UsamaEquinorAFK
Copy link
Contributor Author

/UpdateDatabase

Copy link

github-actions bot commented Feb 9, 2024

👀 Running migration command... 👀

@UsamaEquinorAFK UsamaEquinorAFK force-pushed the InspectionTypeLovemachine branch from e1ea1cd to 522aac6 Compare February 9, 2024 11:30
Copy link

github-actions bot commented Feb 9, 2024

✨ Successfully ran migration command! ✨

@Afonso-2403 Afonso-2403 force-pushed the InspectionTypeLovemachine branch from 522aac6 to 9414ccc Compare February 9, 2024 14:34
@Afonso-2403 Afonso-2403 merged commit 9573881 into equinor:main Feb 9, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working database-change Will require migration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants