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.

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

Fix redirect in add to microsoft365 calendar option #835

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Daniel-Lagos
Copy link

@Daniel-Lagos Daniel-Lagos commented Apr 2, 2024

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

  • [ x ] He revisado que no haya ninguna PR (pull request) ya abierta con un problema similar, siguiendo el apartado de buenas prácticas
  • [ x ] He revisado localmente los cambios para asegurarme de que no haya errores ni problemas.
  • [ x ] He probado estos cambios en múltiples dispositivos y navegadores para asegurarme de que la landing page se vea y funcione correctamente.
  • [ x ] He actualizado la documentación, si corresponde.

Copy link

vercel bot commented Apr 2, 2024

@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.

@Daniel-Lagos Daniel-Lagos changed the title Fix outlook calendar Fix redirect in add to microsoft365 calendar option Apr 2, 2024
@AlejandroSuero
Copy link
Contributor

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
Copy link
Contributor

@Daniel-Lagos

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

@Daniel-Lagos
Copy link
Author

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants