-
Notifications
You must be signed in to change notification settings - Fork 110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cantidad empleos por categoría. #217
Cantidad empleos por categoría. #217
Conversation
…e trabajos que hay disponibles para esta.
{ | ||
<span class="search-job-label-item"><a href="@location.Link">@location.Description</a></span> | ||
} | ||
</p>*@ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Como buenos boys and girl scouts vamos a quitar todos los comments que no hacen nada, como este
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
@@ -14,5 +20,36 @@ protected EmpleoDotNetController() | |||
_database = new EmpleadoContext(); | |||
_uow = new EntityFrameworkUnitOfWork(_database); | |||
} | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No puedes hacer un query por cada categoría, haz una unica consulta con las categorias que deseas con el count
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bien, lo resolveré.
y puedes poner la cantidad entre paréntesis Desarrollo Web (5) |
Cerrado debido a que ya hay otro PR #244 que resuelve esto |
Con esto quedaría resuelto el issue #213.