Codeinwp / intergeo-maps

3 stars 9 forks source link

migrate pro into free #50

Closed selul closed 7 years ago

selul commented 7 years ago

@contactashish13 i would like to stop the working on the pro version and integrate it into the free version. We need to make sure that this will work ok for users who bought the pro one.

after this i would like to make the next changes:

contactashish13 commented 7 years ago

@selul can you please confirm this is what you want in the settings sidebar and the maps sidebar?

settings page

library/maps page

contactashish13 commented 7 years ago

@selul in pirate forms are you talking about this? Its not an ajax based form.

image

selul commented 7 years ago

@contactashish13 regarding the settings page, those 2 btns will just redirect to the maps page, there is no fancy behaviour there, just a simple redirect. Both btns will redirect to the same page . The buttons for the settings page will not be in the sidebar, will have the same placement as the add new is having on the LIbrary page. The sidebar will be the same for both of the pages

Regarding the subscription form, yes that's the one, and would be great if we can make it ajax based ;D

When migrating the options you will need to make sure the pro version will still work with the new free and wont break users website.

contactashish13 commented 7 years ago

@selul PR: https://github.com/Codeinwp/intergeo-maps/pull/51

this will work even if the pro is activated. this will show a nag notice to the user to delete the pro plugin.

selul commented 7 years ago

@contactashish13 can we remove the nag ? if is working fine with both active is fine for me.

contactashish13 commented 7 years ago

@selul done

selul commented 7 years ago

@contactashish13 http://prntscr.com/djslsv what will do this enabled checkbox /

contactashish13 commented 7 years ago

@selul if its unchecked, the imported markers will not be exported

selul commented 7 years ago

@contactashish13 let's remove the checkbox, when they click From Computer will automatically import the markers

contactashish13 commented 7 years ago

@selul https://github.com/Codeinwp/intergeo-maps/pull/52