Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Docs: migrations column-type list is incomplete and "PKs must be Text" contradicts the first-class Uuid type / the NAP schema

Aperta Adatta ai principianti
  • Qualcosa non va?
#72 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
82/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Tranquilla
Stack tecnologico
csharp
Ambito
database, documentation

Direzione di ricerca

Inizia da Website/src/docs/migrations.md e confronta le sue regole di migrazione con SchemaYamlSerializer.ParseType, inclusi i token onDelete accettati. Controlla migrations/schema.yaml per l’uso canonico della chiave primaria. Il lavoro è completato quando la pagina elenca i tipi di colonna e i valori di eliminazione accettati e identifica chiaramente Uuid come il tipo previsto per la chiave primaria.

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

Descrizione

Page

https://dataprovider.tech/docs/migrations/ (Website/src/docs/migrations.md)

A. The column-type list is incomplete

The docs say:

Supported column types include Text, Integer, Real, Blob, Boolean, DateTime, Guid, and vector types on PostgreSQL.

But SchemaYamlSerializer.ParseType accepts many more — including the ones almost every real schema needs and that the canonical NAP migrations/schema.yaml uses: Uuid, Int, BigInt, SmallInt, TinyInt, Json/Jsonb, Date, DateTimeOffset, Decimal(p,s), VarChar(n), NVarChar(n|max), Char(n), Float, Double, Money, Time, Xml, RowVersion, Geometry/Geography, Vector(n).

A reader following the docs literally would not know that DateTimeOffset, Json, Uuid, or Decimal even exist. Please publish the full accepted-token list (it can be generated straight from ParseType).

B. The "PKs must be Text" rule contradicts the first-class Uuid type

The Rules section says:

Primary keys must be UUIDs (Text at the column level, generated by your application).

But there is a first-class Uuid type, and the canonical NAP migrations/schema.yaml uses type: Uuid for every primary key. "Text at the column level" directly contradicts real, recommended usage. Please clarify that Uuid is the intended PK column type.

C. (Minor) onDelete tokens

The example only shows onDelete: NoAction. Worth listing all accepted values (NoAction/No_Action, Cascade, SetNull/Set_Null, SetDefault/Set_Default, Restrict).

These cost real investigation time (reading the serializer source) to resolve while authoring a production schema.

Lingua principale
C#
Stelle
67
Fork
4
Nessuna PR unita negli ultimi 30g

Preparare l'ambiente

Apri in Codespaces

Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.

Come iniziare

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

Altre issue di Nimblesite/DataProvider

Tutte le issue di Nimblesite/DataProvider

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.