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 event time with timezone #1231

Merged
merged 4 commits into from Sep 10, 2021

Conversation

bertrandda
Copy link
Contributor

@bertrandda bertrandda commented Jul 5, 2021

Pull Request check-list

To ensure your Pull Request can be accepted as fast as possible, make sure to review and check all of these items:

  • If your changes affects code, did your write the tests?
  • Are tests passing? (npm test on both front/server)
  • Is the linter passing? (npm run eslint on both front/server)
  • Did you run prettier? (npm run prettier on both front/server)

Description of change

Fix event time in CalDAV service when timezone is associated.

Issue: #1211

@codecov
Copy link

codecov bot commented Jul 5, 2021

Codecov Report

Merging #1231 (5a8df20) into master (64d1bb5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1231   +/-   ##
=======================================
  Coverage   96.55%   96.55%           
=======================================
  Files         669      669           
  Lines        8610     8616    +6     
=======================================
+ Hits         8313     8319    +6     
  Misses        297      297           
Impacted Files Coverage Δ
server/services/caldav/index.js 94.44% <100.00%> (+0.69%) ⬆️
...services/caldav/lib/calendar/calendar.formaters.js 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 64d1bb5...5a8df20. Read the comment docs.

Copy link
Contributor

@Pierre-Gilles Pierre-Gilles left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello! Petite question tu l'as testé avec quelle intégration ?

Je sais pas si c'est lié, mais j'ai essayé de synchroniser un calendrier Google via Caldav, avant ça fonctionnait nickel, et là je n'ai pas tous mes évènements (il manque les évènements récents ou j'avais le problème notamment)

@Pierre-Gilles
Copy link
Contributor

Ok, c'est bizarre je ne les reçois pas de l'API Google... Pourtant j'ai bien reset ma DB locale, et relancé une nouvelle sync fraiche...

@Pierre-Gilles
Copy link
Contributor

Bon ça doit pas être lié, j'ai le même souci sur master! On gère la pagination dans l'intégration Caldav?

@bertrandda
Copy link
Contributor Author

J'ai testé sur Nextcloud et iCloud ça me paraissait bon

@bertrandda
Copy link
Contributor Author

Je viens de tester avec Google Agenda, la synchronisation fonctionne.
Tes anciens évènements sont bien synchronisés ? juste les récents posent problèmes. En effet ça pourrait être trop d'évènements pour la réponse. Je vais me renseigner voir si un système de pagination existe en caldav

@Pierre-Gilles
Copy link
Contributor

Tes anciens évènements sont bien synchronisés ? juste les récents posent problèmes. En effet ça pourrait être trop d'évènements pour la réponse.

Yes, en fait c'est mon compte Google principal, et j'ai des évènements dessus depuis pas mal de temps (2013 un truc comme ça), et les évènements de 2013 à 2015 reviennent sans soucis, mais bizarre, je n'ai pas les évènements de 2021

Pour ça que je me demandais si c'est pas un problème de pagination...

Après, ça n'a rien à voir avec cette PR, c'est plus une remarque générale. Je pense que cette PR fonctionne bien!

Va falloir que je créé des évènements dans un autre calendrier plus "vide" pour pouvoir tester du coup :)

@Pierre-Gilles Pierre-Gilles merged commit f059a8a into GladysAssistant:master Sep 10, 2021
@relativeci
Copy link

relativeci bot commented Sep 10, 2021

Job #64: Bundle Size — 5.68MB (0%).

f059a8a vs 64d1bb5

Changed metrics (1/8)
Metric Current Baseline
Cache Invalidation 0% 49.84%
Changed assets by type (0/7)

No changes


View Job #64 report on app.relative-ci.com

@bertrandda bertrandda deleted the fix/caldav-timezone branch November 23, 2021 21:49
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.

None yet

2 participants