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 missing CRS to project load errors #2813

Merged
merged 2 commits into from
Oct 5, 2023
Merged

add missing CRS to project load errors #2813

merged 2 commits into from
Oct 5, 2023

Conversation

PeterPetrik
Copy link
Contributor

@PeterPetrik PeterPetrik commented Sep 27, 2023

fix #941
fix #2362
related to #2450
related to CU-862ke6jaw

added also test with one missing layer, one layer with invalid CRS and whole project has invalid CRS to test suite.

Screenshot from iPhone

IMG_BB728692FECB-1

Screencast from macOS (desktop)

bad-layers-warning.mov

@PeterPetrik PeterPetrik added the ready-for-testing Ready to be tested by testers crew - passed review by devs label Oct 4, 2023
@tomasMizera tomasMizera merged commit cb4af91 into master Oct 5, 2023
8 checks passed
@tomasMizera tomasMizera deleted the crs_fixes branch October 5, 2023 13:36
@jozef-budac
Copy link

Verified on 2.4.0 RC (Android: 376111 (2.4.0))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-testing Ready to be tested by testers crew - passed review by devs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No warning shown if project has no CRS set Project with undefined CRS can make map/recording misbehave
4 participants