Skip to content

Conversation

@kbr-odoo
Copy link
Contributor

@kbr-odoo kbr-odoo commented Oct 9, 2020

@jcs-odoo Could you please review this PR for me? Thank you in advance! 👍

@kbr-odoo kbr-odoo added this to the 14.0 milestone Oct 9, 2020
Copy link
Contributor

@jcs-odoo jcs-odoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it was a great idea to link to Microsoft's guide rather than explaining everything here. However, it lacks some information to make it clearer.
Also, the structure and titles are missing.

There is already more than enough in this doc to help someone find their way, but I think it could be improved for more clarity.

Finally, since you created a new calendar section, it would make sense to move the google calendar doc to this section as well, in this PR.
crm/optimize/google_calendar_credentials

- plus redirection of google_calendar_credential.rst from CRM to General > Calendars > Google
@kbr-odoo kbr-odoo force-pushed the 14.0-outlook-calendar-kbr branch from 143c373 to 0a6a9f2 Compare October 13, 2020 11:41
@kbr-odoo kbr-odoo requested a review from jcs-odoo October 13, 2020 11:41
@kbr-odoo kbr-odoo requested a review from jubodoo October 14, 2020 09:39
@jubodoo jubodoo merged commit 41d2463 into 14.0 Nov 13, 2020
@jubodoo jubodoo deleted the 14.0-outlook-calendar-kbr branch November 13, 2020 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants