Skip to content
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

Fix update and upload resume #45

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

Conversation

sg-prashanth-raju
Copy link

@sg-prashanth-raju sg-prashanth-raju commented Sep 24, 2024

Script was failing due to deprecated pypdf Api. Changed it to new Api's

@sg-prashanth-raju sg-prashanth-raju marked this pull request as ready for review September 24, 2024 18:55
@navchandar navchandar self-assigned this Sep 26, 2024
@navchandar
Copy link
Owner

can you specify the version of pypdf and so we can lock it in requirements.txt ?

@sg-prashanth-raju
Copy link
Author

can you specify the version of pypdf and so we can lock it in requirements.txt ?

We can lock it to 3.9.0. The docs said not to use the deprecated APIs and suggested different ones. On making those changes script is working properly. Can you please run it once and check if it's working as expected?

Also is the update profile function for changing phone no. working for you. Seems like the UI has changed and It's not able to find the mobile field

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