MDEV-36531 Enable Feedback Plugin for DEB Packages
Create a user facing question during installation step to allow users to choose whether to enable the feedback plugin or not.
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.
Note we have had https://salsa.debian.org/mariadb-team/mariadb-server/-/merge_requests/16 open for a long time. It was waiting for @cvicentiu to confirm what is the correct API URL as there was some confusion which of the alternatives is correct, and perhaps some confirmation that the system actually works end-to-end to be deployed now. Seems there are some stats now at https://mariadb.org/feedback-plugin/ so it is up and running again?
Note we have had https://salsa.debian.org/mariadb-team/mariadb-server/-/merge_requests/16 open for a long time. It was waiting for @cvicentiu to confirm what is the correct API URL as there was some confusion which of the alternatives is correct, and perhaps some confirmation that the system actually works end-to-end to be deployed now. Seems there are some stats now at https://mariadb.org/feedback-plugin/ so it is up and running again?
If you @vuvova or @cvicentiu have time to continue on this, it would be good to have feedback plugin enabled upstream so I can see that it works, what the correct API URL is etc and then follow-up by finalizing https://salsa.debian.org/mariadb-team/mariadb-server/-/merge_requests/16
I plan to have it in the next release, that is, by the end of July.
@ottok , please say if you need it earlier, for example, to prepare or to try it out or to update a merge request in salsa or something
I plan to have it in the next release, that is, by the end of July.
@ottok , please say if you need it earlier, for example, to prepare or to try it out or to update a merge request in salsa or something
No urgency on this one.
thanks
Seems this was merged in https://github.com/MariaDB/server/commit/5d1e485e0a1aa4b98f50439c2dfc70ef3a57651a but using a different version than in this Pull Request.