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

port to python3 #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

port to python3 #10

wants to merge 1 commit into from

Conversation

ayushnawal
Copy link

Tested, not many changes required.

Copy link
Member

@srevinsaju srevinsaju left a comment

Choose a reason for hiding this comment

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

@ayushnawal I have tested this, but I guess this is not a fault of the port, but it keeps spinning / checking for clipart and the journal object save button too doesn't work. Basically, there is no functional usage of this activity. Probably something is broken after some files of sugar changed or this activity was abandoned. We will wait for @quozl 's reply. Probably we need to fix that issue before this PR gets fixed

@ayushnawal
Copy link
Author

@srevinsaju agreed, I was facing it too but I look back closely at the code, and there was nothing to port so I submitted the PR changing exec and running 2to3 😅

@quozl
Copy link

quozl commented Feb 25, 2020

Please fix any functional problems with the activity before porting to Python 3. That's clearly written in our Python 3 Porting Guide as "Quiesce the activity source by making sure the activity works properly before porting, closing any solved issues, merging any pull requests or branches and releasing the last Python 2 version".

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