Add uploaded_by and description fields to notification rules
I maintainer di solito rispondono entro 3 giorni
@liu-kenny ci sta già lavorando.
Dal 8/10/2026.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 68/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Tranquilla
- Stack tecnologico
- rust, typescript
- Ambito
- backend-api-design, full-stack
Direzione di ricerca
Inizia con Rule e RulePayload in rule_structs.rs, quindi segui add_rule, get_all_rules e get_all_rules_with_client_info in rule_controller.rs. Aggiorna i tipi in rules.api.ts e le finestre di dialogo di aggiunta/modifica, quindi sostituisci i segnaposto in rules-table.component.html. Il lavoro è completato quando entrambi i campi vengono mantenuti e restituiti dal backend, uploaded_by usa l’identità del client e le finestre di dialogo e la tabella visualizzano i valori.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
The rules table has "Uploaded By" and "Description" columns that display "-" placeholders. The backend Rule struct doesn't have these fields yet, so the frontend can't show real values.
Acceptance Criteria
- Rule struct in rule_structs.rs has uploaded_by and description fields
- add_rule persists both fields; uploaded_by is set from client identity automatically
- get_all_rules and get_all_rules_with_client_info return both fields
- Frontend table columns display real values instead of "-"
- Add Rule dialog includes a description input
- Edit Rule dialog allows editing description
Proposed Solution
Add optional String fields to the Rule struct and RulePayload in rule_structs.rs. Update add_rule in rule_controller.rs to populate uploaded_by by an optionally client set name on the Notification Rule page or by default the client identifier. On the frontend, update rules.api.ts types, the add/edit dialog forms, and remove the "-" hardcodes in rules-table.component.html.
- Lingua principale
- TypeScript
- Stelle
- 5
- Fork
- 1
- Merge medio
- 2g 20h
- PR unite (30g)
- 48
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
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 Northeastern-Electric-Racing/Argos
-
Remove the landing pageApertaby: ai-assisted frontend good-for: new-member spike
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Northeastern-Electric-Racing/Argos#847 ·
I maintainer di solito rispondono entro 3 giorni
-
quick-view-ownership: Build on the app-wide quick view from #898Forse già presa @bracyw l’ha presa 6 giorni fa. Apertaby: ai frontend spike
Northeastern-Electric-Racing/Argos#904 · 1 assegnatario ·
I maintainer di solito rispondono entro 3 giorni
-
Match the Charging fault display to the BMS critical faults the car publishesForse già presa @SimeonChenchev1 l’ha presa 2 giorni fa. Apertaby: ai dev work frontend
Northeastern-Electric-Racing/Argos#902 · 1 assegnatario ·
I maintainer di solito rispondono entro 3 giorni
-
notification-acks: Track clients in memory and give up when they fall too far behindForse già presa @t-obr-iv l’ha presa 6 giorni fa. Apertabackend by: ai-assisted frontend good-for: new-member spike
Northeastern-Electric-Racing/Argos#892 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 3 giorni
-
notification-acks: Track clients in memory and give up when they stop checking inForse già presa @t-obr-iv l’ha presa 6 giorni fa. Apertabackend by: ai-assisted frontend good-for: new-member spike
Northeastern-Electric-Racing/Argos#891 · 1 commento · 1 assegnatario ·
I maintainer di solito rispondono entro 3 giorni
Tutte le issue di Northeastern-Electric-Racing/Argos
Issue simili
-
bug require-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
expressjs/expressjs.com#2549 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
umami-software/umami#4686 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
[Bug]: Device screenshot fails with 500 on iPhone Duo (Xcode 27.2): serve-sim calls simctl without --displayForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 66/100
pingdotgg/t3code#24900 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
inu-appcenter/UNIDorm-web#411 ·
I maintainer di solito rispondono entro 1 giorno
-
🐛 bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 2 giorni