tksheet icon indicating copy to clipboard operation
tksheet copied to clipboard

BUG: Function 'checkbox' throws error if parameter 'state' is NONE

Open ClaytonCurtis1947 opened this issue 3 years ago • 2 comments

In function 'checkbox':

The test for 'state' being 'normal' or 'disabled' throws an error if 'state' is NONE, which is the =default= for that parameter

Please consider changing if state.lower() in ("normal", "disabled"):

to if (state) and (state.lower() in ("normal", "disabled")):

ClaytonCurtis1947 avatar Jul 21 '22 13:07 ClaytonCurtis1947

Thanks very much for pointing this out!

It should be fixed in 5.4.1 along with the 2nd such occurrence of the same code in the header version of the function.

ragardner avatar Jul 29 '22 12:07 ragardner

Thanks for the reply and apologies for taking so long to respond!

I really like tksheet, which has made it much easier to code a couple of projects!

Three (sorry) questions:

  1. How can I tell which version I have installed, and what do I need to do to update it? (Just pip install it again?)
  2. What is your recommendation for a site or sites to communicate with the tksheet community? I’d like to find a place to ask questions and – hopefully – contribute.
  3. Your GitHub page for tksheet says “”Except for bug fixing I am no longer working on this repository”. Should I be following you elsewhere?

Thanks again.

Clayton Curtis

Newton, MA

From: ragardner @.> Sent: Friday, July 29, 2022 8:03 AM To: ragardner/tksheet @.> Cc: Clayton Curtis @.>; Author @.> Subject: Re: [ragardner/tksheet] BUG: Function 'checkbox' throws error if parameter 'state' is NONE (Issue #126)

Thanks very much for pointing this out!

It should be fixed in 5.4.1 along with the 2nd such occurrence of the same code in the header version of the function.

— Reply to this email directly, view it on GitHub https://github.com/ragardner/tksheet/issues/126#issuecomment-1199197719 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AIINHQ7BQ7RV42UUMITIOC3VWPCA5ANCNFSM54HWSJMA . You are receiving this because you authored the thread. https://github.com/notifications/beacon/AIINHQ6J2CBPGKJERKL6WD3VWPCA5A5CNFSM54HWSJMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOI55E4FY.gif Message ID: @.*** @.***> >

ClaytonCurtis1947 avatar Aug 17 '22 21:08 ClaytonCurtis1947

Hi Clayton, I'm sorry for such a late response,

  1. Lately I have been updating the changelog with this information and you can use pip install tksheet --upgrade to update
  2. There isn't such a community as far as I'm aware, either here or on stackoverflow is best I suppose, you're welcome to open an issue even if just for discussion
  3. The place to follow me at the moment is here on github, I will update my page with links if that changes

Kind regards,

ragardner

ragardner avatar Mar 27 '23 17:03 ragardner

Thanks!

-----Original Message----- From: ragardner @.> To: ragardner/tksheet @.> Cc: Clayton Curtis @.>; Author @.> Sent: Mon, Mar 27, 2023 1:26 pm Subject: Re: [ragardner/tksheet] BUG: Function 'checkbox' throws error if parameter 'state' is NONE (Issue #126)

Closed #126 as completed.— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

ClaytonCurtis1947 avatar Mar 27 '23 17:03 ClaytonCurtis1947