Skip to content

NAS-121621 / 23.10 / remove onedrivesdk (deprecated a long time ago)#11143

Merged
yocalebo merged 1 commit intomasterfrom
NAS-121621
Apr 21, 2023
Merged

NAS-121621 / 23.10 / remove onedrivesdk (deprecated a long time ago)#11143
yocalebo merged 1 commit intomasterfrom
NAS-121621

Conversation

@yocalebo
Copy link
Copy Markdown
Contributor

This module hasn't seen a commit since 2016 so it's dead and it does not work with python3.11 (ships in Cobia). There doesn't seem to be a decent alternative either so we'll drop this for now and revisit if we find another solution.

@yocalebo yocalebo requested a review from a team April 21, 2023 18:55
@bugclerk bugclerk changed the title remove onedrivesdk (deprecated a long time ago) NAS-121621 / 23.10 / remove onedrivesdk (deprecated a long time ago) Apr 21, 2023
@bugclerk
Copy link
Copy Markdown
Contributor

@yocalebo yocalebo merged commit 0041f3a into master Apr 21, 2023
@yocalebo yocalebo deleted the NAS-121621 branch April 21, 2023 22:23
@imkiva
Copy link
Copy Markdown

imkiva commented Nov 5, 2023

Hi all. I just made onedrivesdk compatible with Python 3.11. You can find the repo with my fixes here: https://github.com/imkiva/onedrive-sdk-python and a corresponding Pull Request has been made to the upstream: OneDrive/onedrive-sdk-python#206

I have tested it locally and found it still works just like before. Is it possible to add OneDrive CloudSync back?

@yocalebo
Copy link
Copy Markdown
Contributor Author

yocalebo commented Nov 5, 2023

Hi all. I just made onedrivesdk compatible with Python 3.11. You can find the repo with my fixes here: https://github.com/imkiva/onedrive-sdk-python and a corresponding Pull Request has been made to the upstream: OneDrive/onedrive-sdk-python#206

I have tested it locally and found it still works just like before. Is it possible to add OneDrive CloudSync back?

Hi @imkiva, great job on fixing this! However, we've got no plans on bringing this back. Unless your PR is merged upstream and a new version is published on pypi, I don't see this provider being added again. The last PR that was merged upstream was 7 years ago. I don't expect yours to be merged, unfortunately. Furthermore, this adds support for 3.11 but 3.12 is already released and we will try and keep up to date with the Python versions. Even if we were to bring this back, I suspect new versions of Python will render this library unusable and so we'll be forced, yet again, to remove it.

@ljg-dev
Copy link
Copy Markdown

ljg-dev commented May 23, 2024

Hi all. I just made onedrivesdk compatible with Python 3.11. You can find the repo with my fixes here: https://github.com/imkiva/onedrive-sdk-python and a corresponding Pull Request has been made to the upstream: OneDrive/onedrive-sdk-python#206
I have tested it locally and found it still works just like before. Is it possible to add OneDrive CloudSync back?

Hi @imkiva, great job on fixing this! However, we've got no plans on bringing this back. Unless your PR is merged upstream and a new version is published on pypi, I don't see this provider being added again. The last PR that was merged upstream was 7 years ago. I don't expect yours to be merged, unfortunately. Furthermore, this adds support for 3.11 but 3.12 is already released and we will try and keep up to date with the Python versions. Even if we were to bring this back, I suspect new versions of Python will render this library unusable and so we'll be forced, yet again, to remove it.

And what would you consider acceptable to bring it back if a whole rewrite of the library isn't enough? I don't mean to be disrespectful but if the last time it was updated was 2016, and it worked until now, I don't see any reason why it should stop working in the near future, Python doesn't change that much in short time periods., even if you keep it up to date.

I understand that maybe that feature is not important for TrueNas, but from a user standpoint, it doesn't seem to be right to just remove support for one of the major cloud storage providers there is, without giving us any alternative. How can we be sure that you won't remove support for google drive or S3 tomorrow?

@truenas truenas locked and limited conversation to collaborators May 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants