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

🗓️ Error add to calendar option with microsoft 365 option #831

Closed
Daniel-Lagos opened this issue Apr 1, 2024 · 7 comments
Closed

Comments

@Daniel-Lagos
Copy link

Daniel-Lagos commented Apr 1, 2024

Comportamiento esperado

Se puede agendar el evento de la siguiente manera:

Screenshot 2024-04-01 at 4 49 38 PM

Comportamiento actual

Lanza un error cannot read properties of undefined ( reading 'options')

Screenshot 2024-04-01 at 4 41 03 PM


Información

Estuve revisando y pasa falla cuando el link es de microsoft365, al momento de agendar el evento lanza el error que adjunte en la imagen.

Cambié el dominio cuando viene de microsoft365 por el dominio de outlook (en el archivo /public/js/add-to-calendar.js) y ya funciona bien, sin embargo, quiero saber del porqué no funciona cuando la opción viene microsoft365.

Pasos para reproducirlo

  1. En el home pulsar agregar al calendario
  2. Seleccionar la opción de microsoft365.

Contexto

No uso esa opción jajaja, la uso con google, pero aja, no está mal tenerlo para otras personas que sí lo usen (aclaro porque sino sale mi issue en stream y que esa opción no la usa ni el tato).

Navegador: Arc
Versión: 1.36.0 (48035)
Chromium Engine Version: 123.0.6312.87
SO: MacOS

@AlejandroSuero
Copy link
Contributor

Puedes pasarme el link o el formato del link que tiene que generar?

Que estoy trabajando en esto #483 para poder hacer cambios y estilar de manera más sencilla así como hacer una carga.

@AlejandroSuero
Copy link
Contributor

Es sólo cambiando microsoft365 por outlook?

@Daniel-Lagos
Copy link
Author

Daniel-Lagos commented Apr 2, 2024

Estuve investigando y el problema va más allá de la url, resulta que Microsoft cambio el enrutamiento en mobile, no sé por qué, es un error que deben solucionar según lo que encontré, cambiar la url en el archivo /public/js/add-to-calendar.js no es tan fácil como parece, porque funciona en desktop, pero en mobile no se puede agendar.

@AlejandroSuero
Copy link
Contributor

@Daniel-Lagos de momento en la PR #834 sólo llevo implementados google y outlook para escritorio.

Miraré más adelante cuando funcionen todos los de escritorio, cómo hacer los de móvil.

Que el minificado y anonimato de add-to-calendar.js es complejo de leer con los if raros que hace el anonimato y las variables de una letra 😅

@Daniel-Lagos
Copy link
Author

@AlejandroSuero jajajajaj sii, minificado es supercomplejo de leer, pero ya lo tengo, voy a subir un PR con los ajustes, para que se cree la preview y me ayudes a revisar si funciona en mobile también.

@Daniel-Lagos
Copy link
Author

@AlejandroSuero mira este PR, #835 , pero los previews no se pueden ver y están fallando :c, sabes que puede ser?

@AlejandroSuero
Copy link
Contributor

@AlejandroSuero mira este PR, #835 , pero los previews no se pueden ver y están fallando :c, sabes que puede ser?

Lo de las preview, supongo que @midudev las desactivó excepto para él. Pero no estoy seguro.

@midudev midudev closed this as completed Nov 4, 2024
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

No branches or pull requests

3 participants