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

now TUICalendarOptions properly makes use of all options on the scheduleView and taskView properties #46

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

koga25
Copy link

@koga25 koga25 commented Jul 28, 2022

The properties didn't allow the person to use only one of the settings of the property, if you wanted to show only task/milestone on taskView or only allday/time, it wasn't possible.

Now those properties work as a string[], you can define what settings you want in each one by modifying the array or using the helper functions defined in the files.

@gismofx
Copy link
Owner

gismofx commented Jul 28, 2022

Thanks for working on this! I punted on this one and haven't come back to finish.

I think we need to maintain the taskView and scheduleView enumerated helper types so the api easier/more intuitive for the users... the tricker part is writing the json converters. Would you want to take a go at that? :)

Also, please make your PRs to the dev branch.

…d TUIScheduleView, now it works when assigning a bool value or an string[] value.
@koga25 koga25 changed the base branch from main to dev July 29, 2022 00:47
@koga25
Copy link
Author

koga25 commented Jul 29, 2022

Something like this? I never tried making a json converter before, but it's working. I also included implicit conversions for string[] and bool values to make it equal to the documentation.

Changed to dev branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Idea
Development

Successfully merging this pull request may close these issues.

2 participants