Docs: migrations column-type list is incomplete and "PKs must be Text" contradicts the first-class Uuid type / the NAP schema
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 82/100
- Tipo de issue
- Documentación
- Claridad
- Bien especificado
- Estado de actividad
- Tranquilo
- Stack tecnológico
- csharp
- Área
- database, documentation
Línea de trabajo
Comienza con Website/src/docs/migrations.md y compara sus reglas de migración con SchemaYamlSerializer.ParseType, incluidos los tokens onDelete aceptados. Comprueba migrations/schema.yaml para ver el uso canónico de la clave primaria. Se considera terminado cuando la página enumera los tipos de columna y los valores de eliminación aceptados, e identifica claramente Uuid como el tipo previsto para la clave primaria.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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 (
Textat 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.
- Lenguaje dominante
- C#
- Estrellas
- 67
- Forks
- 4
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de Nimblesite/DataProvider
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
Nimblesite/DataProvider#109 · 2 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
Nimblesite/DataProvider#108 ·
-
A check constraint the schema cannot name can never be retired, and keeps enforcing the old ruleAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
Nimblesite/DataProvider#107 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
Nimblesite/DataProvider#106 ·
-
migrate --allow-destructive drops EVERY foreign key when the database already matches the schemaAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
Nimblesite/DataProvider#105 · 5 comentarios ·
Todos los issues de Nimblesite/DataProvider
Issues similares
-
:watch: Not Triaged dotnet-target-version
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
copilot documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 2 días
-
untriaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
dotnet/dotnet-api-docs#13124 ·
Los mantenedores suelen responder en 1 día
-
agentic-workflows
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
type:bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
BHoM/MidasCivil_Toolkit#441 ·