Skip to content

Comments

Plugin installation#125

Merged
3nids merged 8 commits intomainfrom
plugin-installation
Feb 10, 2026
Merged

Plugin installation#125
3nids merged 8 commits intomainfrom
plugin-installation

Conversation

@domi4484
Copy link
Member

@domi4484 domi4484 commented Feb 6, 2026 •
edited
Loading

  1. Click Install
    • If running oQtopus from QGIS, the plugin is installed in the current QGIS profile.
    • If running oQtopus standalone, you get asked to select the QGIS profile directory.

@github-actions
Copy link

github-actions bot commented Feb 6, 2026 •
edited
Loading

🔌 Plugin Artifact

A development build of the plugin is available for testing:

⬇️ Download oqtopus_dev.zip

⚠️ This is a development build from commit 037dc75.

@domi4484 domi4484 force-pushed the plugin-installation branch from 202ad4d to d7a3ba1 Compare February 6, 2026 08:22
@domi4484 domi4484 requested a review from 3nids February 6, 2026 09:24
3nids
3nids reviewed Feb 6, 2026
View reviewed changes
3nids
3nids reviewed Feb 6, 2026
View reviewed changes
3nids
3nids approved these changes Feb 6, 2026
View reviewed changes
Copy link
Member

@3nids 3nids left a comment

Choose a reason for hiding this comment

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

nice!

@domi4484
Copy link
Member Author

domi4484 commented Feb 6, 2026

nice!

I just have one remaining small issue, I don't now how to detect if there where errors during installation. In QGIS it does a pushMessage, i guess I can connect to that somehow:

        level = Qgis.MessageLevel.Success if success else Qgis.MessageLevel.Critical
        iface.pluginManagerInterface().pushMessage(msg, level)

@3nids
Copy link
Member

3nids commented Feb 6, 2026

qgis/QGIS#64848

@3nids
Copy link
Member

3nids commented Feb 6, 2026

I don't think you can easily connect to this otherwise.
I would say that it's fine that il will be better in upcoming release.
You could check if the plugin is installed afterwards? (via API)

@domi4484 domi4484 force-pushed the plugin-installation branch from 2c3dd11 to a25401e Compare February 6, 2026 15:28
domi4484
domi4484 commented Feb 6, 2026
View reviewed changes
@ponceta
Copy link
Collaborator

ponceta commented Feb 10, 2026

Testing Plugin Install :

image image image

@ponceta
Copy link
Collaborator

ponceta commented Feb 10, 2026

Current QGIS profile and Current plugin version could be great to have indeed :

image

Upgrade of an older plugin version works too.

When installed from a zip artefact, oqtopus gets corrupted after each reboot of QGIS. (mentionned in #130)

Otherwise LGTM!

@domi4484
Copy link
Member Author

domi4484 commented Feb 10, 2026

@ponceta I have added the missing informations if you want to test again

@ponceta
Copy link
Collaborator

ponceta commented Feb 10, 2026

I run a quick test and we can merge this!

domi4484 added a commit to teksi/TMMT that referenced this pull request Feb 10, 2026
@domi4484
Pass iface for plugin installation
008b8d6 
Needs opengisch/oqtopus#125
@domi4484 domi4484 mentioned this pull request Feb 10, 2026
Closed
@ponceta
Copy link
Collaborator

ponceta commented Feb 10, 2026

image

Good for me, tested with 0e47074

3nids
3nids reviewed Feb 10, 2026
View reviewed changes
oqtopus/gui/plugin_widget.py Outdated

self.__current_module_package = None
self.__plugin_name = None
self.__iface = qgis_iface
Copy link
Member

@3nids 3nids Feb 10, 2026

Choose a reason for hiding this comment

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

another simpler approach is to simply import iface
https://github.com/osgeosuomi/flake8-qgis?tab=readme-ov-file#qgs105

Copy link
Member Author

@domi4484 domi4484 Feb 10, 2026

Choose a reason for hiding this comment

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

Thanks I'll use that

domi4484
domi4484 commented Feb 10, 2026
View reviewed changes
@3nids 3nids merged commit 57301fd into main Feb 10, 2026
7 checks passed
@3nids 3nids deleted the plugin-installation branch February 10, 2026 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@3nids 3nids 3nids approved these changes

+1 more reviewer

@ponceta ponceta ponceta left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@domi4484 @3nids @ponceta