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

Generate deltas no matter the vector QFS layer action #4242

Closed
wants to merge 2 commits into from

Conversation

suricactus
Copy link
Collaborator

Removed QFieldCloudUtils::isCloudAction as an obsolete method

@m-kuhn for testing

@qfield-fairy
Copy link
Collaborator

qfield-fairy commented Apr 28, 2023

@nirvn nirvn marked this pull request as draft May 11, 2023 04:52
suricactus and others added 2 commits May 27, 2023 16:31
Removed `QFieldCloudUtils::isCloudAction` as an obsolete method
@nirvn
Copy link
Member

nirvn commented May 27, 2023

@suricactus , my comment was actually wrong, the code was bailing out in addLayerListeners() if the project wasn't a cloud one. I got confused as the cloud project check happens earlier when we handle the QgsProject::onHomePathChanged. I've moved the cloud check to occur in LayerObserver::onLayersAdded to mimic what we do in LayerObserver::onHomePathChanged.

I think this is ready to merge.

@nirvn nirvn marked this pull request as ready for review June 4, 2023 04:56
@nirvn
Copy link
Member

nirvn commented Jun 4, 2023

@suricactus , @m-kuhn , can we merge this?

@m-kuhn
Copy link
Member

m-kuhn commented Jun 4, 2023

Several people mentioned bad results (empty deltas) with this. Please hold back.

@nirvn
Copy link
Member

nirvn commented Jun 4, 2023

@m-kuhn , roger that.

@nirvn nirvn closed this Jul 15, 2023
@m-kuhn m-kuhn reopened this Jul 15, 2023
@nirvn
Copy link
Member

nirvn commented Jul 15, 2023

@m-kuhn , you beat me to it, was closing/re-opening to catch latest CI naming changes :)

@suricactus
Copy link
Collaborator Author

Several people mentioned bad results (empty deltas) with this. Please hold back.

@m-kuhn Do you have steps to reproduce?

@m-kuhn
Copy link
Member

m-kuhn commented Aug 18, 2023

@SeqLaz and @lucienicolier should know more

@m-kuhn m-kuhn closed this Aug 18, 2023
@m-kuhn m-kuhn reopened this Aug 18, 2023
@nirvn
Copy link
Member

nirvn commented Dec 2, 2023

@suricactus , should we close this PR or is it still relevant?

@suricactus
Copy link
Collaborator Author

Will close the PR, but there is still work to be done here.

@suricactus suricactus closed this Dec 3, 2023
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.

5 participants