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

separate storage on QField and QFC #393

Merged
merged 7 commits into from
Nov 3, 2023
Merged

separate storage on QField and QFC #393

merged 7 commits into from
Nov 3, 2023

Conversation

lucienicolier
Copy link
Member

No description provided.

Copy link
Collaborator

@suricactus suricactus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have added a bit of intro for the qfc storage.

documentation/how-to/projects.en.md Outdated Show resolved Hide resolved
documentation/get-started/storage-qfc.en.md Show resolved Hide resolved
lucienicolier and others added 3 commits October 27, 2023 15:24
Co-authored-by: Ivan Ivanov <suricactus@users.noreply.github.com>
Co-authored-by: Ivan Ivanov <suricactus@users.noreply.github.com>
@nirvn
Copy link
Member

nirvn commented Oct 27, 2023

@lucienicolier
Copy link
Member Author

Will this link in QField need updating: https://github.com/opengisch/QField/blob/master/src/qml/QFieldLocalDataPickerScreen.qml#L567 ?

Good point! Yes, it will.

@nirvn
Copy link
Member

nirvn commented Oct 27, 2023

I'm wondering if we should avoid changing the URL for QField storage when splitting the cloud part out. We've used that crucial link quite a lot when helping ppl deal with Android storage. Ppl on social media linked to it too.

@lucienicolier
Copy link
Member Author

I'm wondering if we should avoid changing the URL for QField storage when splitting the cloud part out. We've used that crucial link quite a lot when helping ppl deal with Android storage. Ppl on social media linked to it too.

I can keep the one for QField and only change the one for QFC

@nirvn
Copy link
Member

nirvn commented Oct 27, 2023

@lucienicolier , that sounds like a better option to me, thanks.

@suricactus suricactus merged commit aaa37e8 into master Nov 3, 2023
1 of 2 checks passed
@suricactus suricactus deleted the storage branch November 3, 2023 22:36
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.

4 participants