rtCamp / rtMedia

rtMedia (formerly Buddypress Media) adds advanced multimedia functionality to WordPress, BuddyPress and bbPress
https://rtmedia.io/
122 stars 110 forks source link

Deactivation Reason: collecting user data while deactiving the plugin #1934

Closed thebengalboy closed 1 year ago

thebengalboy commented 2 years ago

Description We want to collect feedback when the plugin is deactivated on the site.

What we will collect:

Procedure:

Feedback Title: If you have a moment, please let us know why you are deactivating: Feedback options:

Example Prototype: https://prnt.sc/xnqHwpmctoLj

robicse11127 commented 2 years ago

@chandrapatel @gagan0123 After doing RND to find out how we can implement this feature into our plugin. I found out most of the plugins who are using this feature are using service from Freeminus or Appsero or somethings similar to this. Who provides a SDK to integrate in the plugin or theme and track down the survey data with the popup and getting the used feedback when deactivating. So will we you the same for us? Please let me know your thoughts. Thanks. Cc: @thebengalboy

thebengalboy commented 2 years ago

Please note that we may not be able to afford a paid service for the time being. As a result, we need to seek an appropriate free choice or develop our own.

thebengalboy commented 2 years ago

@robicse11127 Is there any drawback to the method suggested by @chandrapatel if we do not use a third-party service? What are the additional benefits of having a 3rd party service?

robicse11127 commented 2 years ago

The benefit of using those services are...

And if we want to create from the scratch

thebengalboy commented 2 years ago

@robicse11127 We can not go with Appsero, because it'll require moving the whole release process there. And I do not see Freeminus has a free package that can serve us. So, I assume building it on our own is the last option.

robicse11127 commented 1 year ago

@pavanpatil1 @pooja-muchandikar Here we need to install the rt-plugin-survey plugin in the rtmedia.io site to have the stored survey data. Also in this plugin we have a settings page where we need to add the administrator Username and Application Password to be saved.

image

The Application Password can be generated from the the users profile showing in the below image.

image

And in the rtMedia plugin we also need to update the API URL with the https://rtmedia.io

image
pooja-muchandikar commented 1 year ago

@robicse11127 Thanks for the details!

@pavanpatil1 @Pradeep1308 please check this if its working as expected

pooja-muchandikar commented 1 year ago

@pavanpatil1 please cross check the functionality using rtMedia Core & use the plugin on rtMedia.io dev site and check if everything is working as expected.

cc: @aviral-mittal

pavanpatil1 commented 1 year ago

Hi @robicse11127, verified the PR https://github.com/rtCamp/rtMedia/pull/1939, not able to submit the deactivation form. After clicking on submit button the console error is displayed. Also, there are a few conflicts in PR could you please check ?.

https://user-images.githubusercontent.com/44057535/215652393-925875ac-c9ee-4e05-a256-d3ba8bf569c7.mp4

robicse11127 commented 1 year ago

It is been tested and code conflict merged.

kjnanda commented 1 year ago

@pavanpatil1 Please check my comment below.

Please check this visual -

https://github.com/rtCamp/rtMedia/assets/15173772/9d3d155c-3ebb-479e-8380-13abfd6985d9

Please check this snapshot - image

Utsav-Ladani commented 1 year ago

Hello @kjnanda,

I disable the HTTP authentication to test this feature on the dev site. Please enable it back after the testing. I also added some code changes to handle the error in this PR.

kjnanda commented 1 year ago

@Utsav-Ladani Kindly get this PR reviewed first. As after review it would need another round of testing.

Also, regarding http auth, I'm not clear why do we need to remove it?

CC @pavanpatil1

Utsav-Ladani commented 1 year ago

@kjnanda this PR is reviewed now. you can test it.

We need to disable HTTP auth for testing because the rtmedia-dev.rtm.rt.gw site is protected with HTTP auth for security reasons, and our plugin deactivation survey is sent to this site. So if we want to test the plugin, we need to specify the password in the request, or we can test it by disabling it for a moment.

However, on the actual site rtmedia.io, we don't need anything because it doesn't have HTTP auth.

kjnanda commented 1 year ago

@Utsav-Ladani This is working as expected now.

kjnanda commented 1 year ago

Cross verified wrt dev branch and added fix working properly.

kjnanda commented 1 year ago

Tested after release and it works as expected.