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.

Sign up for GitHub

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

Jump to bottom

Would like to remove caching #72

Open
SPlanzer opened this issue Sep 15, 2020 · 2 comments
Open

Would like to remove caching #72

SPlanzer opened this issue Sep 15, 2020 · 2 comments

Comments

@SPlanzer
Copy link

Caching was added to the application because the WMS server was taking many minutes to responded to the getCapabilities request.

Now WMS has been turned off it would be nice to remove the extra complexity the caching added to the source code.

As a first step, tests will need to be undertaken to ensure WFS and WMTS documents are served to the app fast enough as not to block the application. If docs take to long the app will appear frozen while waiting as requests are in the main thread.
Potentially all requests could be moved to another thread.
@strk strk mentioned this issue Sep 16, 2020
Closed
@strk
Copy link
Contributor

strk commented Sep 16, 2020

Does this deprecates #40 ?

@SPlanzer
Copy link
Author

Does this deprecates #40 ?

#40 should be closed as now WMS is no longer supported it is not required

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@strk @SPlanzer