Skip to content

configuración y gestión de CRUD OLIMPIADAS - #85

Open
FatimaZahraCode wants to merge 4 commits into
2DAW-CarlosIII:masterfrom
FatimaZahraCode:segundaConvocatoria
Open

configuración y gestión de CRUD OLIMPIADAS#85
FatimaZahraCode wants to merge 4 commits into
2DAW-CarlosIII:masterfrom
FatimaZahraCode:segundaConvocatoria

Conversation

@FatimaZahraCode

Copy link
Copy Markdown

No description provided.

public function index()
{
$cursos = Curso::with('edicion')->get();
return $cursos;

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.

Has creado app/Http/Resources/CursoResource.php pero no lo utilizas para la devolución de los datos.

if (!$curso) {
return response()->json(['message' => 'Curso no encontrado'], 404);
}
return response()->json($curso);

@albsierra albsierra Jun 1, 2026

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.

JSON se devuelve en los endpoints asociados a las API, pero no cuando lo vamoss a utilizar con Blade

Lo que se pretende es hacer el CRUD de cursos en el dashboard, por lo que deberías haber creado este controlador fuera de Api/v1.

El único endpoint que se pedía era /api/v1/cursos

Comment thread app/Models/Edicion.php Outdated
}
public function cursos()
{
return $this->hasMany(Curso::class , 'edicion_id', 'id');

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.

A cada edición solo le corresponde un curso, por lo que se trataría de una relación hasOne

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.

Los cursos que deberías haber tomado de referencia deberían haber sido los de la página de las olimpiadas.

Además, has creado database/factories/CursoFactory.php que no has utilizado

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants