Skip to content

Remove Office 365 Protocol and documentation #1163

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

RogerSelwyn
Copy link
Contributor

Office 365 api is no linger available from Microsoft, so this PR is to remove it. If you disagree, then feel free to reject.

I've also change mention of Office 365, (where it does not relate to the Office 365 API), to Microsoft 365 in line with current MS branding.

@alejcas
Copy link
Member

alejcas commented May 7, 2025

ok. It's true Office 365 must be removed as it's deprecated.
But I want to also reflect that O365 can be implemented with other apis like microsoft sharepoint rest api or ms dynamics rest api.

Can you not change the tittle of the library or at least leave something like:

"A simple python library to interact with Microsoft Graph and other MS api"

@RogerSelwyn
Copy link
Contributor Author

ok. It's true Office 365 must be removed as it's deprecated. But I want to also reflect that O365 can be implemented with other apis like microsoft sharepoint rest api or ms dynamics rest api.

Can you not change the tittle of the library or at least leave something like:

"A simple python library to interact with Microsoft Graph and other MS api"

The other api I can see in the connection.py is MS Business Central 365, which I don't really know anything about. I assume this is Dynamics. I'm happy to go through and mention it into the docs in places. I'm not sure which objects it supports access to and there isn't anything in the docs that mentions it to go on so it would be very light.

You should be able to commit to this PR as well to improve it.

@RogerSelwyn
Copy link
Contributor Author

Or feel free to reject it, if it isn't setting the right impression, I'm very relaxed about it :-)

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.

2 participants