elmau/mauflix
elmau
/
mauflix
2
1
Fork 0
Code Issues 3 Pull Requests Releases Wiki Activity

Habilitación de búsqueda y API 😎 #6

Closed
perro wants to merge 8 commits from master into develop
pull from: master
merge into: elmau:develop
Conversation 7 Commits 8 Files Changed - +1117 -576
perro commented 2023-01-20 17:53:52 -06:00
Collaborator
Copy Link

Hice los cambios que me encomendaste:

  • Fuente a local.
  • Adición de logo.
  • Arreglo de despliegue de la ficha de cada peli (donde añadí el tooltip en hover)

Además de esos cambios, ya quedó completamente habilitada la búsqueda (fotos adjuntas).

Como consecuencia colateral, resultó que habilitar la API eran menos de 20 líneas de código adicionales, así que eso también ya quedó :D (fotos adjuntas)

Hice una nueva sección de ayuda donde explica cómo sacarle el jugo a la búsqueda y a la API (foto adjunta)

Deshabilité la sección para reportar problemas. Quisiera hacerla como app separada donde el formulario detecte la URL problemática y en el panel de administración haya un área de "bugs" con ese reporte. Principalmente pa practicar eso de separación de apps.

Para echarlo andar se requiere habilitar la extensión unaccent en psql, en nuestros enlaces compartidos puedes ver cómo.

En la API, cuando el usuario no está autenticado, se elimina el campo file_name. Sin embargo, con esto me entró la duda de si los usuarios no autenticados pueden abrir directamente la URL de ese campo.

Saludos!

Hice los cambios que me encomendaste: * Fuente a local. * Adición de logo. * Arreglo de despliegue de la ficha de cada peli (donde añadí el tooltip en hover) Además de esos cambios, ya quedó completamente habilitada la búsqueda (fotos adjuntas). Como consecuencia colateral, resultó que habilitar la API eran menos de 20 líneas de código adicionales, así que eso también ya quedó :D (fotos adjuntas) Hice una nueva sección de ayuda donde explica cómo sacarle el jugo a la búsqueda y a la API (foto adjunta) Deshabilité la sección para reportar problemas. Quisiera hacerla como app separada donde el formulario detecte la URL problemática y en el panel de administración haya un área de "bugs" con ese reporte. Principalmente pa practicar eso de separación de apps. Para echarlo andar se requiere habilitar la extensión unaccent en psql, en nuestros enlaces compartidos puedes ver cómo. En la API, cuando el usuario no está autenticado, se elimina el campo `file_name`. Sin embargo, con esto me entró la duda de si los usuarios no autenticados pueden abrir directamente la URL de ese campo. Saludos!
Captura de pantalla 2023-01-20 - 15.49.04.png
135 KiB
Captura de pantalla 2023-01-20 - 15.51.47.png
62 KiB
Captura de pantalla 2023-01-20 - 15.39.35.png
50 KiB
Captura de pantalla 2023-01-20 - 15.51.40.png
55 KiB
Screenshot 2023-01-20 at 15-41-45 MauFlix.png
296 KiB
Captura de pantalla 2023-01-20 - 15.49.04.png Captura de pantalla 2023-01-20 - 15.51.47.png Captura de pantalla 2023-01-20 - 15.39.35.png Captura de pantalla 2023-01-20 - 15.51.40.png Screenshot 2023-01-20 at 15-41-45 MauFlix.png
perro added 2 commits 2023-01-20 17:53:53 -06:00
elmau commented 2023-01-20 23:23:57 -06:00
Owner
Copy Link

pero... ¿Estabas trabajando en naster?

pero... ¿Estabas trabajando en `naster`?
perro commented 2023-01-23 19:49:22 -06:00
Author
Collaborator
Copy Link

Ey, ya revisé.

Pues tuve que haber trabajado en una rama cualquiera a partir de develop y después pedir el PR a develop. Sin embargo, lo que hice fue trabajar directamente en develop.

Puedes ignorar este PR y probar la rama develop xD

Lo que veo también es que la master tiene dos commits hace un año que no estaban en develop, mucho antes de que empezará a moverle al mauflix…

Ey, ya revisé. Pues tuve que haber trabajado en una rama cualquiera a partir de `develop` y después pedir el PR a `develop`. Sin embargo, lo que hice fue trabajar directamente en `develop`. Puedes ignorar este PR y probar la rama `develop` xD Lo que veo también es que la `master` tiene dos commits hace un año que no estaban en `develop`, mucho antes de que empezará a moverle al mauflix…
elmau commented 2023-01-23 21:15:10 -06:00
Owner
Copy Link

pero lo extraño es que dice master to develop. 😆

vale, pruebo develop... directamente...

pero lo extraño es que dice `master` to `develop`. 😆 vale, pruebo develop... directamente...
elmau closed this pull request 2023-01-23 21:15:16 -06:00
perro commented 2023-01-24 18:00:21 -06:00
Author
Collaborator
Copy Link

jahajaja ya veo. Eso pasa cuando haces un pull request con urgencia. Upssss xD

jahajaja ya veo. Eso pasa cuando haces un pull request con urgencia. Upssss xD
elmau commented 2023-01-24 19:03:23 -06:00
Owner
Copy Link

pero... no tenemos prisa... 😆😆😆

pero... no tenemos prisa... 😆😆😆
perro commented 2023-01-24 19:33:57 -06:00
Author
Collaborator
Copy Link

Yo sí, para irme de vacas jajahajaj

Yo sí, para irme de vacas jajahajaj
elmau commented 2023-01-24 19:41:48 -06:00
Owner
Copy Link

¿Otra vez?... 😆

¿Otra vez?... 😆

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: elmau/mauflix#6
No description provided.
Delete Branch "master"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?