Replace NewCascadeDeleter positional arguments with a config struct

Aperta

#1,848 aperta il 5/8/2026 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà: 3/5
Tempo stimato: 1-2 giorni
Idoneità per principianti: 68/100

Tipo di issue: Refactoring
Chiarezza: Abbastanza chiara
Stato di attività: Tranquilla
Stack tecnologico: go
Ambito: backend

Direzione di ricerca

Inizia individuando NewCascadeDeleter e lo stack di pulizia org-delete introdotto in PR #1838. Leggi i suoi chiamanti e il package deleter per identificare tutte le 17 dipendenze e i relativi punti di costruzione. Sostituisci la costruzione posizionale con la struct delle dipendenze denominata, quindi esegui i test Go pertinenti e conferma che il progetto venga compilato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

NewCascadeDeleter takes 17 positional dependencies after the org-delete cleanup stack (#1838). Every new dependency is appended positionally, which is easy to get wrong since several arguments share the same interface shape.

Replace the positional list with a struct (e.g. deleter.Deps) so construction is named and order-independent.

Raised in review: https://github.com/raystack/frontier/pull/1838#discussion_r3711259260

🤖 Generated with Claude Code

Lingua principale: Go
Stelle: 344
Fork: 47
Merge medio: 4g 4h
PR unite (30g): 26

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

Leggi tutta la issue e poi la guida ai contributi del progetto.
Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
Fai un fork del repository e lavora su un branch.
Apri una pull request che faccia riferimento al numero della issue.

Altre issue di raystack/frontier

Share one postgres container across the internal/store/postgres test suites Aperta

enhancement go

Difficoltà 4/5 3-5 giorni Idoneità per principianti 62/100

raystack/frontier#1932 · Aperta il 17/9/2026
Store the invoice amount_remaining so local rows can tell what is still owed Aperta

Difficoltà 4/5 3-5 giorni Idoneità per principianti 58/100

raystack/frontier#1907 · Aperta il 28/8/2026
UpdatePermission renames instead of updating metadata (and drops the metadata it is given) Aperta

authz bug

Difficoltà 3/5 1-2 giorni Idoneità per principianti 70/100

raystack/frontier#1896 · 1 commento · Aperta il 19/8/2026
Add OpenTelemetry tracing Aperta

Difficoltà 5/5 Più di una settimana Idoneità per principianti 45/100

raystack/frontier#1875 · Aperta il 12/8/2026
Disabled orgs are still granted permissions through the Check API Aperta

Difficoltà 5/5 Più di una settimana Idoneità per principianti 38/100

raystack/frontier#1863 · Aperta il 7/8/2026

Tutte le issue di raystack/frontier

Issue simili

nix: vendorHash is outdated Aperta

Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100

abenz1267/elephant#301 · 1 reazione · Aperta il 24/9/2026
Bob Shell support Aperta

enhancement

Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100

rossoctl/cortex#1119 · Aperta il 24/9/2026
[BUG] CREATE property value with *, -, ^, % or unary minus is stored as its text ('2 * 3'); + and / are evaluated Aperta

bug

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

orneryd/NornicDB#656 · Aperta il 24/9/2026
HaveExactElements' failure message omits where the missing or extra elements start when that index is 0 Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

onsi/gomega#934 · Aperta il 24/9/2026
propertyNames and contentSchema errors: InstanceLocation is overwritten by later siblings Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

santhosh-tekuri/jsonschema#276 · Aperta il 24/9/2026

Altre issue su Go