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

Rebase and merge with TurtleArt and other changes related to Port #8

Merged
merged 24 commits into from Aug 29, 2020

Conversation

Saumya-Mishra9129
Copy link
Member

@Saumya-Mishra9129 Saumya-Mishra9129 commented May 28, 2020

  • Rebase and merge with TurtleArt

  • Port from GStreamer Gst.Message.structure to get_structure()

  • Port to Python3

  • Port to GTK3

  • Port from GConf to GioSettings

Fixes #4, Fixes #1.
@quozl @pro-panda Please review.

@Saumya-Mishra9129
Copy link
Member Author

Getting this error while testing

Screenshot from 2020-06-10 01-20-04

@srevinsaju @chimosky can you review please.

@srevinsaju
Copy link
Member

srevinsaju commented Jun 9, 2020

@Saumya-Mishra9129 as the error suggests, there is no *.gschema .xml file as there is on turtleart-activity.

@Saumya-Mishra9129
Copy link
Member Author

Saumya-Mishra9129 commented Jun 9, 2020 via email

@srevinsaju
Copy link
Member

Ok; I will review the diff tomorrow morning too. (got held up with some work)

plugins/camera_sensor/tacamera.py Outdated Show resolved Hide resolved
turtleflags.py Outdated Show resolved Hide resolved
@chimosky
Copy link
Member

Reviewed, thanks.

As this is an update of upstream changes, all your commit messages should reflect that.

@Saumya-Mishra9129
Copy link
Member Author

@quozl @chimosky @JuiP @srevinsaju , Please review and test.

@srevinsaju
Copy link
Member

Ok as I can't do a huge diff review, I will do an activity testing :D

@srevinsaju
Copy link
Member

@Saumya-Mishra9129 what is the test procedure for this activity? I was not able to click on any of the flags, does it use any key combination to activate them?

@Saumya-Mishra9129
Copy link
Member Author

Saumya-Mishra9129 commented Aug 17, 2020

@Saumya-Mishra9129 what is the test procedure for this activity? I was not able to click on any of the flags, does it use any key combination to activate them?

No, Actually there were some minor bugs in the code. Fixed them. @chimosky @quozl @srevinsaju Please test and merge.

turtleflags.py Outdated Show resolved Hide resolved
@Saumya-Mishra9129
Copy link
Member Author

@chimosky I have tested activity works fine. Please merge.

@chimosky
Copy link
Member

Reviewed, thanks.

Collaboration seems to be broken for the activity, although this PR has nothing to do with it but if you have the time you can fix it in a new PR. Thanks.

@chimosky chimosky merged commit 79ac22c into sugarlabs:master Aug 29, 2020
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.

Rebase and merge with Turtle Art Port from GConf to Gio.Settings
5 participants