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

refresh events every 5 minutes #132

Closed
wants to merge 2 commits into from
Closed

refresh events every 5 minutes #132

wants to merge 2 commits into from

Conversation

georgehrke
Copy link
Member

partial fix for #31

@georgehrke georgehrke added the 2. developing Work in progress label Oct 17, 2016
@mention-bot
Copy link

@georgehrke, thanks for your PR! By analyzing the history of the files in this pull request, we identified @tcitworld, @MathieuSchopfer and @raghunayyar to be potential reviewers.

@coveralls
Copy link

coveralls commented Oct 17, 2016

Coverage Status

Coverage decreased (-0.03%) to 42.857% when pulling 999cb7a on refresh_events into 4a6cd85 on master.

@tcitworld
Copy link
Member

Why not PROPFIND too every 5 minutes ?

@georgehrke
Copy link
Member Author

Note to myself - there is one more issue:

  • click existing event to edit it
  • try to drag it -> won't work because its locked
  • wait for refresh to kick in
  • dragging now suddenly works -> that's not how it's supposed to be

Possible solution:

  • take fcEvent that's supposed to be edited
  • remove it from calendar
  • use renderEvent with stick=true to render as individual event

@coveralls
Copy link

coveralls commented Oct 17, 2016

Coverage Status

Coverage decreased (-0.03%) to 42.857% when pulling e3036cf on refresh_events into 4a6cd85 on master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 42.857% when pulling e3036cf on refresh_events into 4a6cd85 on master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 42.857% when pulling e3036cf on refresh_events into 4a6cd85 on master.

@georgehrke
Copy link
Member Author

@tcitworld PROPFIND takes twice the requests.

Also when it comes to refreshing calendars, it'd like to keep the original object from the initial request and not replace it every 30 seconds ^^

@tcitworld
Copy link
Member

Rebased.

@georgehrke
Copy link
Member Author

Needs a proper implementation in the vue app + cdav-library

@georgehrke georgehrke closed this Nov 21, 2018
@georgehrke georgehrke deleted the refresh_events branch November 21, 2018 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants