Dear Julien, That’s well noted. Thank you. I will add a detailed description in each PR in the future. Regarding the issue of LICENSE file requirements, I totally agree with you. I will also ask Tim if he has suggestions about it.
Best regards, Lova — [Image] Lova Andriarimalala QGIS Full Stack Developer Visit http://kartoza.com<http://kartoza.com/> to find out about open source: * Desktop GIS programming services * Geospatial web development * GIS Training * Consulting Services Office: +261(0)34 09 524 73<tel:+261340952473> From: Julien Moura <[email protected]> Date: Friday, 24 November 2023 at 10:47 AM To: Lova Andriarimalala <[email protected]>, [email protected] <[email protected]> Subject: Re: [QGIS-Developer] QGIS Full Stack Web Developer Report Thanks for your quick reply here Lova, I've no special legibility that my opinion would be greater than yours, so I can't tell if it has to be reverted or not. That's why I ask for discussion before deploying some breaking change, without any warning or information. After having looked to others PR, I would like to see more description about what a PR does exactly, especially when it breaks something. In this case, even after the last PR, this is a breaking and undocumented change. For now, a plugin developer has no way to see that a LICENSE file is now required in its plugin's zip. An error message in a log is not a suitable information, even it's well formulated and clear, especially in the era of automated CI/CD deployments. Regards, Julien On 24/11/2023 08:35, Lova Andriarimalala wrote: Dear Julien, Many thanks for your feedback. In the new PR, the license file is only required for new plugins. For existing plugin updates, it generates just a warning (but doesn't fail) when the license file is missing. However, I'm not sure if we should also just generate a warning for new plugin uploads for now. If so, I will also fix the new plugin upload. Kind regards. — [Image] Lova Andriarimalala QGIS Full Stack Developer Visit http://kartoza.com<http://kartoza.com/> to find out about open source: * Desktop GIS programming services * Geospatial web development * GIS Training * Consulting Services Office: +261(0)34 09 524 73<tel:+261340952473> From: QGIS-Developer <[email protected]><mailto:[email protected]> on behalf of Julien Moura via QGIS-Developer <[email protected]><mailto:[email protected]> Date: Friday, 24 November 2023 at 10:30 AM To: [email protected]<mailto:[email protected]> <[email protected]><mailto:[email protected]> Subject: Re: [QGIS-Developer] QGIS Full Stack Web Developer Report Hello Lova, I cross post my comment to this issue<https://github.com/qgis/QGIS-Django/issues/38#issuecomment-1824010198> related to the PR mentioned below as "Make LICENSE file as required in plugin package<https://github.com/qgis/QGIS-Django/pull/309>", because I did not have any answer there but saw that some changes still have been applied without any comment. While trying to publish or update a plugin, we faced the new error message related to the deployment of this PR<https://github.com/qgis/QGIS-Django/pull/309>: > Fault string: <Fault 1: 'File upload must be a valid QGIS Python plugin > compressed archive. Cannot find LICENSE in plugin package.'> See downstream issue on qgis-plugin-ci project (disclaimer: I'm one of the mainteners but speaking on my own here): https://github.com/opengisch/qgis-plugin-ci/issues/255 I think this kind of change, which breaks the plugins'publication flow, should be discussed before to be implemented (an issue from 1 person seems to be too light to decide without any discussion), announced to the community, a warning campaign should be run and a transitional phase should be implemented (warning for 6 months, then error). This has a direct impact on hundreds (thousands?) of plugin developers on a community project with several million end users. I understand that this process may seem too cumbersome, and that since the QGIS Django project hasn't been so dynamic for a few years, it's nice to see it get a new lease of life, even if it means merging and deploying on an ongoing basis. As for the underlying principle, I'm generally in favor of strengthening the control mechanisms (automatic or otherwise) for extensions on the official repository, but I think it's really important to do this gradually, or at least to avoid unilateral change "descended from the skies of the developers". Concerning the idea of integrating the license in the plugin package, I'm not really convinced of the interest since most plugins are contaminated by the GPL2+ of QGIS <-- Qt and the license is never displayed to the end user. But why not. After all, it's always a good practice to include licence and spread the word about (re)usage rules. Reverting sounds maybe too rought so I suggest modyfing the behavior to lower the level and make it a simple warning and in the meanwhile starting a communication and preventive work upstream: 1. update documentation: https://docs.qgis.org/3.28/en/docs/pyqgis_developer_cookbook/plugins/plugins.html 2. communicate on the QGIS Dev list **before** the implementation to discuss the rationale 3. integrate a warning mechanism 4. manage the QGIS versions concerned (only applicable to new QGIS released versions after this being merged) A last question: did you have some pre-production environment where to deploy new changes in order to evaluate them before publishing widely? Or some versioning logic, milestone workflow where PRs are grouped before being deployed? Regards, Julien On 17/11/2023 13:59, Lova Andriarimalala via QGIS-Developer wrote: Hello everyone, Please find below the report summarizing the progress on the feed and plugins website development for this week. PRs open: 1. Add support for renaming plugin name<https://github.com/qgis/QGIS-Django/pull/302> 2. Add command to fix none in search results<https://github.com/qgis/QGIS-Django/pull/304> 3. Show more records, records items per page<https://github.com/qgis/QGIS-Django/pull/306> 4. Specify tag page title and other plugin page title<https://github.com/qgis/QGIS-Django/pull/308> 5. Make LICENSE file as required in plugin package<https://github.com/qgis/QGIS-Django/pull/309> PR merged: 6. Update dockerfile and requirements for production<https://github.com/qgis/qgis-feed/pull/44> 7. Update requirements according to production<https://github.com/qgis/qgis-feed/pull/47> 8. Add geoip2 in production, setting up log<https://github.com/qgis/qgis-feed/pull/48> 9. Use contry code when testing daily_visit.country<https://github.com/qgis/qgis-feed/pull/51> Still working on: 1. Fresh plugin includes obsolete stuff <https://github.com/qgis/QGIS-Django/issues/46> Changes to the QGIS Feed website are now deployed and available at https://feed.qgis.org. Have a great weekend, Lova — [Image] Lova Andriarimalala QGIS Full Stack Developer Visit http://kartoza.com<http://kartoza.com/> to find out about open source: * Desktop GIS programming services * Geospatial web development * GIS Training * Consulting Services Office: +261(0)34 09 524 73<tel:+261340952473> From: Lova Andriarimalala <[email protected]><mailto:[email protected]> Date: Friday, 10 November 2023 at 5:32 PM To: [email protected]<mailto:[email protected]> <[email protected]><mailto:[email protected]> Subject: Re: QGIS Full Stack Web Developer Report Hello everyone, Please find below the report summarizing the progress on the feed and plugins website development for this week. PRs open: 10. Add support for renaming plugin name<https://github.com/qgis/QGIS-Django/pull/302> 11. Update dockerfile and requirements for production<https://github.com/qgis/qgis-feed/pull/44> PR merged: 1. Add web page UI on the root URL<https://github.com/qgis/qgis-feed/pull/43> Still working on: 2. Deployment of the feed updates Have a great weekend, Lova — [Image] Lova Andriarimalala QGIS Full Stack Developer Visit http://kartoza.com<http://kartoza.com/> to find out about open source: * Desktop GIS programming services * Geospatial web development * GIS Training * Consulting Services Office: +261(0)34 09 524 73<tel:+261340952473> From: Lova Andriarimalala <[email protected]><mailto:[email protected]> Date: Friday, 3 November 2023 at 5:05 PM To: [email protected]<mailto:[email protected]> <[email protected]><mailto:[email protected]> Subject: Re: QGIS Full Stack Web Developer Report Hello everyone, Please find below the report summarizing the progress on the feed site development for this week. PRs open: 1. Add web page UI on the root URL<https://github.com/qgis/qgis-feed/pull/43> PR merged: 1. Using rich editor for content, check data validity<https://github.com/qgis/qgis-feed/pull/42> Still working on: 1. Some checks and fixes on the entirety<https://github.com/qgis/qgis-feed/issues/18> Have a great weekend, Lova — [Image] Lova Andriarimalala QGIS Full Stack Web Developer Visit http://kartoza.com<http://kartoza.com/> to find out about open source: * Desktop GIS programming services * Geospatial web development * GIS Training * Consulting Services Office: +261(0)34 09 524 73 From: Lova Andriarimalala <[email protected]><mailto:[email protected]> Date: Friday, 27 October 2023 at 4:53 PM To: [email protected]<mailto:[email protected]> <[email protected]><mailto:[email protected]> Subject: Re: QGIS Full Stack Web Developer Report Hello everyone, I'm grateful for your warm welcome, and I'm truly thrilled to join this wonderful community. Here is the report summarizing the progress on the feed site development for this week. PRs open: * Form submission management<https://github.com/qgis/qgis-feed/pull/39> PR merged: * Feeds list page with filter and sorting<https://github.com/qgis/qgis-feed/pull/30> * Feed item form with preview<https://github.com/qgis/qgis-feed/pull/32> * Feed form review step<https://github.com/qgis/qgis-feed/pull/33> * Implement webpack and use it for Bulma CSS<https://github.com/qgis/qgis-feed/pull/34> Still working on: * Improve feed item form: Check data validity<https://github.com/qgis/qgis-feed/issues/25> Have a great weekend, Lova — [cid:9A13FDC2-2DFE-48D9-9736-E7E67663BF60] Lova Andriarimalala QGIS Full Stack Web Developer Visit http://kartoza.com<http://kartoza.com/> to find out about open source: * Desktop GIS programming services * Geospatial web development * GIS Training * Consulting Services Office: +261(0)34 09 524 73 From: Lova Andriarimalala <[email protected]><mailto:[email protected]> Date: Friday, 20 October 2023 at 2:54 PM To: [email protected]<mailto:[email protected]> <[email protected]><mailto:[email protected]> Subject: QGIS Full Stack Web Developer Report Hello everyone, Currently, I am working on the website that administers the feeds displayed in the news area of QGIS. The website's repository is available on GitHub: https://github.com/qgis/qgis-feed Please find below the report for the past two weeks. PRs open: * Feeds list page with filter and sorting<https://github.com/qgis/qgis-feed/pull/30> * Feed item form with preview<https://github.com/qgis/qgis-feed/pull/32> * Feed form review step<https://github.com/qgis/qgis-feed/pull/33> PR merged: * Create a login page, add test cases, configure GH actions<https://github.com/qgis/qgis-feed/pull/28> Still working on: * Use web pack for bulma CSS<https://github.com/qgis/qgis-feed/issues/29> Please do not hesitate to provide any feedback regarding the report structure or layout. Your suggestions are highly appreciated, as they will enable us to improve the report's quality and readability. Thank you for taking the time to review the report. Have a great weekend, Lova — [cid:E4BD0DDE-7267-4E95-BE4E-92E12A6851D3] Lova Andriarimalala QGIS Full Stack Web Developer Visit http://kartoza.com<http://kartoza.com/> to find out about open source: * Desktop GIS programming services * Geospatial web development * GIS Training * Consulting Services Office: +261(0)34 09 524 73 _______________________________________________ QGIS-Developer mailing list [email protected]<mailto:[email protected]> List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
_______________________________________________ QGIS-Developer mailing list [email protected] List info: https://lists.osgeo.org/mailman/listinfo/qgis-developer Unsubscribe: https://lists.osgeo.org/mailman/listinfo/qgis-developer
