Skip to content

[raft/scd] Extract delete opintent - #1654

Merged
mickmis merged 1 commit into
interuss:masterfrom
Orbitalize:scd_extract_delete_opintent
Sep 2, 2026
Merged

mickmis merged 1 commit into
interuss:masterfrom
Orbitalize:scd_extract_delete_opintent

Conversation

@MariemBaccari

@MariemBaccari MariemBaccari commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Extract delete opintent business logic.

Implements #1701

@MariemBaccari
MariemBaccari marked this pull request as ready for review August 20, 2026 09:26
@MariemBaccari
MariemBaccari force-pushed the scd_extract_delete_opintent branch from be0e4cf to 44bfcb3 Compare August 20, 2026 09:29
@barroco barroco added the dss-raft Relating to the application-layer consensus implemenation based on raft label Aug 20, 2026
@MariemBaccari
MariemBaccari force-pushed the scd_extract_delete_opintent branch 2 times, most recently from 30cec77 to 25270fa Compare August 20, 2026 15:29
@MariemBaccari
MariemBaccari force-pushed the scd_extract_delete_opintent branch from 25270fa to 6613cf6 Compare August 28, 2026 10:09

@mickmis mickmis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

(6613cf6#r3895361616) LGTM

Comment thread pkg/scd/actions/operational_intents.go Outdated
@MariemBaccari
MariemBaccari force-pushed the scd_extract_delete_opintent branch 2 times, most recently from 3d46207 to a404ca1 Compare September 2, 2026 08:24
@MariemBaccari
MariemBaccari force-pushed the scd_extract_delete_opintent branch from a404ca1 to 52cd0b1 Compare September 2, 2026 11:38
@mickmis
mickmis merged commit 06ca2da into interuss:master Sep 2, 2026
18 checks passed
@mickmis
mickmis deleted the scd_extract_delete_opintent branch September 2, 2026 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dss-raft Relating to the application-layer consensus implemenation based on raft

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants