Skip to content

Correctly document compatibility rules for QGIS 4 plugin migration#288

Merged
Xpirix merged 2 commits intoqgis:masterfrom
kannes:improve-qgis4-migration
Apr 2, 2026
Merged

Correctly document compatibility rules for QGIS 4 plugin migration#288
Xpirix merged 2 commits intoqgis:masterfrom
kannes:improve-qgis4-migration

Conversation

@kannes
Copy link
Copy Markdown
Contributor

@kannes kannes commented Mar 31, 2026

Meant as alternative to #277 but I am not sure if it really is an improvement. I changed a lot as I got deeper and deeper into the document.

Ideally there would be just one single canonical page with all the information for code migration AND compatibility settings.

I probably broke Django templating but hope there is a CI process that will give me a preview 😬

Copy link
Copy Markdown
Collaborator

@Xpirix Xpirix left a comment

Choose a reason for hiding this comment

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

Thanks so much @kannes . It's clearer now.
Just a few suggestions to not break the Django template, otherwise, LGTM.

Here is the preview of this change:

Image

Comment thread qgis-app/templates/flatpages/docs_migrate_qgis4.html Outdated
Comment thread qgis-app/templates/flatpages/docs_migrate_qgis4.html Outdated
Comment thread qgis-app/templates/flatpages/docs_migrate_qgis4.html Outdated
Comment thread qgis-app/templates/flatpages/docs_migrate_qgis4.html Outdated
Co-authored-by: Lova Andriarimalala <43842786+Xpirix@users.noreply.github.com>
@kannes
Copy link
Copy Markdown
Contributor Author

kannes commented Apr 1, 2026

Yay, thank you!

@Xpirix
Copy link
Copy Markdown
Collaborator

Xpirix commented Apr 2, 2026

LGTM, thanks so much @kannes !

@Xpirix Xpirix merged commit 1a3beec into qgis:master Apr 2, 2026
2 checks passed
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