OrganiseDesktop icon indicating copy to clipboard operation
OrganiseDesktop copied to clipboard

Fixed tkinter and local imports

Open Sory-Noroc opened this issue 3 years ago • 6 comments

Hey! I am a new contributor, also new to Git, so I would appreciate if you would accept me in this project, and perhaps guide me as I develop/contribute to the app. :)

Sory-Noroc avatar Mar 13 '21 07:03 Sory-Noroc

Hey @Sory-Noroc welcome to the crew. :) I will take a look at your changes on Sunday evening and accept them if they are good, but I think they will be. thank you for helping out.

blavejr avatar Mar 13 '21 15:03 blavejr

First of all, Thank you for your contribution.

if sys.version_info >= (3,) works already well in Python2. I doubt replacing it to try statement is a good idea. If it didn't work before, please tell us.

other than that- everything looks good. blavejr will accept it as soon as he checks the code.

KokoseiJ avatar Mar 14 '21 23:03 KokoseiJ

I agree with @KokoseiJ, the try might not be the best solution unless there is another reason.

blavejr avatar Mar 15 '21 20:03 blavejr

You might be right but I was getting an ImportError with the previous version from that tkMessageBox

Lun, 15 mar. 2021, 22:56 Remigius Kalimba Jr @.***> a scris:

I agree with @KokoseiJ https://github.com/KokoseiJ, the try might not be the best solution unless there is another reason.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/blavejr/OrganiseDesktop/pull/93#issuecomment-799746341, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOWHLJ4NHMQBGXMUGYMILVTTDZYA3ANCNFSM4ZDR2UCQ .

Sory-Noroc avatar Mar 15 '21 20:03 Sory-Noroc

You might be right but I was getting an ImportError with the previous version from that tkMessageBox Lun, 15 mar. 2021, 22:56 Remigius Kalimba Jr @.***> a scris: I agree with @KokoseiJ https://github.com/KokoseiJ, the try might not be the best solution unless there is another reason. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#93 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOWHLJ4NHMQBGXMUGYMILVTTDZYA3ANCNFSM4ZDR2UCQ .

Hmmm are you running this on python3? because in that case we might be missing something

blavejr avatar Mar 15 '21 21:03 blavejr

Yes, ofc Im running it on python3

Lun, 15 mar. 2021, 23:05 Remigius Kalimba Jr @.***> a scris:

You might be right but I was getting an ImportError with the previous version from that tkMessageBox Lun, 15 mar. 2021, 22:56 Remigius Kalimba Jr @.***> a scris: … <#m_8996315261215186939_> I agree with @KokoseiJ https://github.com/KokoseiJ https://github.com/KokoseiJ, the try might not be the best solution unless there is another reason. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#93 (comment) https://github.com/blavejr/OrganiseDesktop/pull/93#issuecomment-799746341>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOWHLJ4NHMQBGXMUGYMILVTTDZYA3ANCNFSM4ZDR2UCQ .

Hmmm are you running this on python3? because in that case we might be missing something

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/blavejr/OrganiseDesktop/pull/93#issuecomment-799751789, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOWHLJ5EMRT5GSNQ4OAVGG3TDZZCJANCNFSM4ZDR2UCQ .

Sory-Noroc avatar Mar 15 '21 21:03 Sory-Noroc