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 an auto-refresh feature to GeoMOOSE #89

Closed
theduckylittle opened this issue Jun 29, 2015 · 8 comments
Closed

Add an auto-refresh feature to GeoMOOSE #89

theduckylittle opened this issue Jun 29, 2015 · 8 comments
Assignees
Milestone

Comments

@theduckylittle
Copy link
Member

Well... let's spec this out after I wrote the code:

<map-source ... auto-refresh="5000">
 ...
</map-source>

The auto-refresh attribute is specified in milliseconds.

@klassenjs
Copy link
Member

Looks like it works to me. My only comment is that it causes the layer to flash when it reloads. This is worse when the Loading.js extension is used as the whole map dims under the loading spinny circle thing. Is there something we could do with the OL transitions to avoid this?

klassenjs added a commit to geomoose/geomoose-docs that referenced this issue Jun 30, 2015
@theduckylittle
Copy link
Member Author

@klassenjs -- Maybe. I'm willing to call this a wrap for 2.8 and if someone wants something smoother but it together for a 2.8.1 or 2.9 release. Do you want to file an enhancement ticket?

Also, I think if the refresh rate is reasonable (I've been testing 5 seconds, reasonable = 15s?), it'll be less noticeable. Right now, however, we're using a mechanism that is very reliable and integrated reasonably well into OL.

@klassenjs
Copy link
Member

Ok.
On Jun 30, 2015 11:49, "theduckylittle" notifications@github.com wrote:

@klassenjs https://github.com/klassenjs -- Maybe. I'm willing to call
this a wrap for 2.8 and if someone wants something smoother but it together
for a 2.8.1 or 2.9 release. Do you want to file an enhancement ticket?

Also, I think if the refresh rate is reasonable (I've been testing 5
seconds, reasonable = 15s?), it'll be less noticeable. Right now, however,
we're using a mechanism that is very reliable and integrated reasonably
well into OL.


Reply to this email directly or view it on GitHub
#89 (comment).

@kc0bqb
Copy link

kc0bqb commented Jul 13, 2015

I am using Geomoose 2.7.1 and I do not have a Layer.js file. Is there a newer release?

Josh

@theduckylittle
Copy link
Member Author

Not yet. We are working on 2.8 and getting really close. All the tickets
should be in review status by today and many are being tested for release
already.
On Jul 13, 2015 8:41 AM, "kc0bqb" notifications@github.com wrote:

I am using Geomoose 2.7.1 and I do not have a Layer.js file. Is there a
newer release?

Josh


Reply to this email directly or view it on GitHub
#89 (comment).

@theduckylittle
Copy link
Member Author

This is even documented! And tested! Yay!

@kc0bqb
Copy link

kc0bqb commented Jul 17, 2015

If I replace the MapSource.js and add the Layer.js to the GeoMoose folder will it work? If not what else do I have to change to implement this feature?

Joshua Meyer

Lafayette County Sheriff's Office GIS

107 S 11th St

Lexington, MO 64067

Phone: 660-259-2796

Cell: 660-259-1234

Fax: 660-259-2545

From: theduckylittle [mailto:notifications@github.com]
Sent: Friday, July 17, 2015 12:52 PM
To: geomoose/geomoose
Cc: kc0bqb
Subject: Re: [geomoose] Add an auto-refresh feature to GeoMOOSE (#89)

This is even documented! And tested! Yay!


Reply to this email directly or view it on GitHub #89 (comment) . https://github.com/notifications/beacon/AMsuMnMIIV150gFvuaOpA26ndLC3oHx6ks5oeTg3gaJpZM4FOe1x.gif

@klassenjs
Copy link
Member

There were lots of internal changes when Layer.js was introduced. You can
either track them down and cherry-pick them with git, download the nightly
build and help us test, or wait a bit for the 2.8 release.
On Jul 17, 2015 3:25 PM, "kc0bqb" notifications@github.com wrote:

If I replace the MapSource.js and add the Layer.js to the GeoMoose folder
will it work? If not what else do I have to change to implement this
feature?

Joshua Meyer

Lafayette County Sheriff's Office GIS

107 S 11th St

Lexington, MO 64067

Phone: 660-259-2796

Cell: 660-259-1234

Fax: 660-259-2545

From: theduckylittle [mailto:notifications@github.com]
Sent: Friday, July 17, 2015 12:52 PM
To: geomoose/geomoose
Cc: kc0bqb
Subject: Re: [geomoose] Add an auto-refresh feature to GeoMOOSE (#89)

This is even documented! And tested! Yay!


Reply to this email directly or view it on GitHub <
https://github.com/geomoose/geomoose/issues/89#issuecomment-122355664> . <
https://github.com/notifications/beacon/AMsuMnMIIV150gFvuaOpA26ndLC3oHx6ks5oeTg3gaJpZM4FOe1x.gif>


Reply to this email directly or view it on GitHub
#89 (comment).

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