Gabriel - Segunda convocatoria OlimpiadasC3 - #81
Conversation
|
Efectivamente, el primer ejercicio no modificaba ningún archivo de los que rastrea |
| use App\Models\Edicion; | ||
| use Illuminate\Database\Seeder; | ||
|
|
||
| class CursoSeeder extends Seeder |
There was a problem hiding this comment.
Muy bien solucionado el seeder de cursos.
| </li> | ||
| </ul> | ||
| <p>La siguiente es la relación de cursos en las que se han publicado los ejercicios de las últimas ediciones:</p> | ||
| @php |
There was a problem hiding this comment.
Ya existe un método convertirARomano en app/Models/Categoria.php.
Lo más conveniente habría sido reutilizar este método, quizás refactorizándolo como un helper, ya que, según lo has programado, el próximo curso, el administrador debería tocar el código para generar el 18.
| Route::get('/', function () { | ||
| return view('welcome'); | ||
| // Sacamos las ediciones para que aparezcan en blade | ||
| $ediciones = \App\Models\Edicion::with('curso')->orderBy('curso_escolar', 'desc')->get(); |
There was a problem hiding this comment.
Aunque este código sea muy pequeño, routes/web.php no es el lugar para colocarlo. Siempre lo hemos hecho en un controlador.
Una alternativa habría sido:
Route::get('/', function () {
return view('welcome', compact(\App\Models\Edicion::withCurso()));
})->name('home');
Y crear ese método withCurso() en el modelo de Edicion.
No obstante, lo mejor habría sido crear un componente para los cursos de Moodle. En Olimpiadas hay varios ejemplos.
…sC3 into segundaConvocatoria
iborra0306
changed the title
2 participants
No description provided.