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

Code improvements for Different CRS #290

Merged
merged 2 commits into from
Jul 4, 2017

Conversation

lucernae
Copy link
Collaborator

@lucernae lucernae commented Jul 4, 2017

BBOX will be validated by qgis_server helpers.
- Convert bbox to EPSG:4326 for displaying layers
- Try to retrieve layer projection and save Layer CRS
- Standardize thumbnail generation to use EPSG:4326
@codecov
Copy link

codecov bot commented Jul 4, 2017

Codecov Report

Merging #290 into master-qgis_server will increase coverage by 0.06%.
The diff coverage is 65.51%.

Impacted file tree graph

@@                  Coverage Diff                   @@
##           master-qgis_server     #290      +/-   ##
======================================================
+ Coverage               45.72%   45.79%   +0.06%     
======================================================
  Files                     261      261              
  Lines                   19009    19044      +35     
  Branches                 2498     2500       +2     
======================================================
+ Hits                     8692     8721      +29     
- Misses                   9824     9827       +3     
- Partials                  493      496       +3
Impacted Files Coverage Δ
geonode/qgis_server/views.py 47.22% <0%> (-0.45%) ⬇️
geonode/layers/tests.py 97.36% <100%> (ø) ⬆️
geonode/qgis_server/helpers.py 67.39% <100%> (+1.09%) ⬆️
geonode/layers/views.py 40.26% <100%> (+0.93%) ⬆️
geonode/qgis_server/tasks/update.py 73.91% <100%> (+0.57%) ⬆️
geonode/qgis_server/signals.py 81.11% <72.22%> (-0.99%) ⬇️
geonode/layers/utils.py 48.91% <0%> (+0.21%) ⬆️

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 0450e95...1df6b2e. Read the comment docs.

@lucernae lucernae merged commit 462aba6 into kartoza:master-qgis_server Jul 4, 2017
@lucernae lucernae deleted the code_improvements branch July 4, 2017 16:48
@lucernae lucernae changed the title Code improvements Code improvements for Different CRS Jul 5, 2017
@lucernae
Copy link
Collaborator Author

lucernae commented Jul 5, 2017

Also made some improvements for #284 . Cover cases when FileField wrongly assumes that QML style doesn't exists

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.

Layer map view does not zoom to layer correctly Thumbnails not showing for some datasets
1 participant