midudev / la-velada-web-oficial

Web oficial de La Velada IV de Ibai Llanos
https://lavelada.es
Other
1.39k stars 609 forks source link

Fix redirect in add to microsoft365 calendar option #835

Closed Daniel-Lagos closed 6 days ago

Daniel-Lagos commented 7 months ago

Descripción

Se cambió la estructura de la url para crear eventos con microsoft365, ya que Microsoft está enrutando de manera diferente en mobile, agregue una solución temporal, mientras Microsoft soluciona eso. Se agregó en el archivo add-to-calendar.js la validación si es mobile para cambiar la url de redireccionamiento para crear un evento.

Problema solucionado

Se le da solución al issue Error add to calendar option with microsoft 365 option

Tomo como referencia una solución propuesta en el siguiente repositorio add-to-calendar-button

Cambios propuestos

  1. Validar el dispositivo y asignar la url que corresponde
  2. Como es un archivo minificado es complicado ver el cambio; sin embargo, agrego acá la línea que se modificó "outlook" == t ? "outlook.live.com" : "outlook.office.com", a += isAndroid() || isiOS() ? "/calendar/0/deeplink/compose?path=%2Fcalendar%2Faction%2Fcompose&rru=addevent" : "/calendar/action/compose?rru=addevent"

Comprobación de cambios

vercel[bot] commented 7 months ago

@Daniel-Lagos is attempting to deploy a commit to the midudev pro Team on Vercel.

A member of the Team first needs to authorize it.

AlejandroSuero commented 7 months ago

He probado los cambios.

Sin actualizar el dominio de outlook, te pide registrarte.

link generado actualmente

Screenshot 2024-04-02 at 11 37 55

link con el dominio cambiado outlook.office.com -> outlook.live.com

Screenshot 2024-04-02 at 11 38 19
AlejandroSuero commented 7 months ago

@Daniel-Lagos

Por si se quiere cambiar ese comportamiento también o no.

Daniel-Lagos commented 7 months ago

@AlejandroSuero creo que una vez te registres ya solo te pide el correo, pero sí, ese comportamiento está feo, voy a revisar más tarde a ver que encuentro, gracias por tu feedback.