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

Updated logic to save download location in QgsSettings #51

Merged
merged 1 commit into from
Jul 4, 2023

Conversation

ronitjadhav
Copy link
Collaborator

@ronitjadhav ronitjadhav commented Jul 3, 2023

#48
Store the download location as a QgsSetting
Set the default to ~/Downloads
Update it when the user changes the download location when downloading other resources
Don't update the download location when the user doesn't download it (e.g. add model to QGIS directly)

@ronitjadhav ronitjadhav requested a review from ismailsunni July 3, 2023 10:00
Copy link
Collaborator

@ismailsunni ismailsunni left a comment

Choose a reason for hiding this comment

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

LGTM

@ismailsunni ismailsunni merged commit fde0021 into main Jul 4, 2023
@ronitjadhav ronitjadhav deleted the store_download_location_in_qgssettings branch July 5, 2023 05:50
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.

2 participants