Skip to content
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.

Sign up for GitHub

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

Test Claudio Baque #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Test Claudio Baque #4

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions test-laravel.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@
publicación que se hizo y que usuario la realizó. Explica si el código a continuación es correcto.
*/

Respuesta: El codigo expuesto solo obtendrá un posts(ya que consultar por ID), lo ideal seria consultar por fecha para obtener todos los registros y/o posts.


/* ### 1. A user made a post yesterday, today a system administrator needs to see the content of the post that was
made and which user made it Explain if the code below is correct.
*/
Expand Down Expand Up @@ -46,6 +49,9 @@ public function show($id)
de registros sea mínimo 10000. Explica si el código a continuación es correcto.
*/

Respuesta: 1.- Falta validar que sea minino 10000 registros.
2.- Falta lógica para considerar tiempo de ejecución

/* ### 2. A developer needs to provide a solution to a request, which must load certain information.
This information is a set of records with the following fields: User_Id, Tittle_Post, Comment.
It is necessary to consider the execution time, the user who performs the port exists and the number of records is at least 10000.
Expand Down Expand Up @@ -88,6 +94,9 @@ public function setData(Request $request){
agregar para que este funcione.
*/

Respuesta: 1.- Falta declarar una variable y asignarle a la Clase ImportAlerts
2.- Mostar en el response la variable a declarar.

/* ### 3. You need to download a file with the system alerts generated by the users,
the number of alert records exceeds 6k records. Explain the code below and what you need to adding to make it work.
*/
Expand Down