Skip to content

chore: Refactor Mounts#1236

Merged
notAreYouScared merged 3 commits intomainfrom
chore/MountNodePivot
May 9, 2025
Merged

chore: Refactor Mounts#1236
notAreYouScared merged 3 commits intomainfrom
chore/MountNodePivot

Conversation

@rmartinoscar
Copy link
Member

@rmartinoscar rmartinoscar commented Apr 11, 2025

Move mount_node, server_mount & egg_mount into a single model_has_mounts table

!eggs && nodes = available to all servers with associated nodes regardless of eggs
eggs && !nodes = available to all servers with associated eggs regarless of nodes
!eggs && !nodes = available to all servers regardless of eggs or nodes

lancepioch
lancepioch previously approved these changes Apr 12, 2025
@rmartinoscar rmartinoscar force-pushed the chore/MountNodePivot branch from 46affe2 to e94a682 Compare April 12, 2025 11:14
@rmartinoscar rmartinoscar marked this pull request as draft April 12, 2025 12:50
@rmartinoscar rmartinoscar force-pushed the chore/MountNodePivot branch from e94a682 to 3b5db45 Compare April 12, 2025 22:21
@rmartinoscar rmartinoscar marked this pull request as ready for review April 12, 2025 22:24
@rmartinoscar rmartinoscar requested a review from lancepioch April 12, 2025 22:25
@rmartinoscar rmartinoscar changed the title chore: Use Pivot instead of Model for MountNode chore: Refactor Mounts Apr 12, 2025
@rmartinoscar rmartinoscar marked this pull request as draft April 12, 2025 22:43
@rmartinoscar rmartinoscar force-pushed the chore/MountNodePivot branch from 3b5db45 to 23c7707 Compare April 24, 2025 18:16
@rmartinoscar rmartinoscar marked this pull request as ready for review April 24, 2025 19:33
@notAreYouScared notAreYouScared requested a review from lancepioch May 3, 2025 15:58
@notAreYouScared notAreYouScared merged commit 7971dc1 into main May 9, 2025
25 checks passed
@notAreYouScared notAreYouScared deleted the chore/MountNodePivot branch May 9, 2025 17:18
@github-actions github-actions bot locked and limited conversation to collaborators May 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants