salesagility / SuiteDocs

SuiteCRM - Open source CRM for the world
https://docs.suitecrm.com/
Other
59 stars 284 forks source link

Add "Disabling Google Sync" section #640

Closed hammerill closed 4 months ago

hammerill commented 4 months ago

As descripted in here: https://github.com/salesagility/SuiteDocs/issues/639

pgorod commented 4 months ago

Thank you for this :+1:

serhiisamko091184 commented 4 months ago

Hey @Hammerill,

thanks for contributing to SuiteCRM project,

I'd like to check if there's a particular reason to 'delete the task'.

image

Is that possible to achieve the same result with just the deactivation of the task? image

image

Thanks in advance.

Regards, Serhii

hammerill commented 4 months ago

Hello @serhiisamko091184, And indeed, there was no particular reason to delete the task, that was my error. It should be Disable (or delete) as at the end.

serhiisamko091184 commented 4 months ago

Thanks for your quick response and update @Hammerill!

If there's no need to delete it, maybe it is better to focus on Disabling / Enabling (or Activating / Deactivating)?

It's more convenient to have the task handy if you want to use it again, rather you need to create the same one later on.

What are your thoughts?

Thanks!

Regards, Serhii

hammerill commented 4 months ago

That's right @serhiisamko091184, there's no need in deleting these tasks, thank you. I will remove any trace of that.

serhiisamko091184 commented 4 months ago

Thank you @Hammerill!

serhiisamko091184 commented 4 months ago

@Hammerill, thanks one more time,

after testing the suggested changes, allow me to share some ideas:

1) To make it consistent with other content, perhaps we could proceed with the same table format (+ code snippet)?

image

2) The title of the task is 'Google Calendar Sync', it's better also to use the same title for the sake of consistency image

3) It's more common to use 'authorisation' as the UK related instead of 'authorization'.

I wonder, is this something you could consider and add to your fix?

Thanks in advance for your cooperation!

Regards, Serhii

hammerill commented 4 months ago

Hello @serhiisamko091184 , thank you for these suggestions, I've made the section more consistent.

serhiisamko091184 commented 4 months ago

Hey @Hammerill,

thanks for your cooperation and quick changes!

Regards, Serhii