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

Add auto-refresh functionality #9

Open
pascalberger opened this issue May 9, 2016 · 3 comments
Open

Add auto-refresh functionality #9

pascalberger opened this issue May 9, 2016 · 3 comments

Comments

@pascalberger
Copy link
Contributor

It would be nice if the PR view could auto-refresh itself (like eg. dashboards could). Use case is if PR view is eg. used on a team screen.

@sierpinski
Copy link
Owner

Sounds like a useful feature.

@karlingen
Copy link
Contributor

This is a fairly straightforward implementation.
You can simply put a <meta http-equiv="refresh" content="30"> inside the <head> of the html files. The iframe will automatically refresh.
30 is the number of seconds before a refresh will occur.

Question to @pascalberger and @sierpinski:

  1. Should this functionality be configurable? As in you can turn the auto-refresh off and on.
  2. Should the number of seconds be configurable, or should it be a static number of say 600 (10 minutes)?

@sierpinski
Copy link
Owner

My opinion:
1 and 2 should be yes, if technically feasible. Your code suggestion seems like it will make this easier.

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

No branches or pull requests

3 participants