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

VTodo and VJournal Missing Setters and Getters #65

Open
spslater opened this issue Mar 29, 2023 · 4 comments
Open

VTodo and VJournal Missing Setters and Getters #65

spslater opened this issue Mar 29, 2023 · 4 comments
Assignees

Comments

@spslater
Copy link
Contributor

The VEvent component has most (all?) of the methods to be able to set and get the different parts of the component like the summary, description, start time, ect. It also has a way to create new events.

The VTodo and VJournal only have a way to serialize and load the information. (I think same is true with VBusy and VTimezone, but I don't need those for my project right now, so I haven't looked at them yet)

If these haven't been implemented yet because there hasn't been a need for it, then I will go ahead and work on adding this functionality.

@arran4
Copy link
Owner

arran4 commented Mar 29, 2023

You're correct:

these haven't been implemented yet because there hasn't been a need for it

Please do, happy to accept PRs. If possible comment, but consistency with the others I think is key right now.

@quite
Copy link
Contributor

quite commented Dec 7, 2023

Yeah, I began using golang-ical for a small project, and creating a VCAL with a VTODO component would be nice, and being able to set all props.

@arran4
Copy link
Owner

arran4 commented Dec 7, 2023

I'm currently travelling but I will see why this has stalled in a couple weeks if not sooner.

@arran4 arran4 linked a pull request Dec 26, 2023 that will close this issue
@arran4
Copy link
Owner

arran4 commented Oct 16, 2024

I believe this was partially or full resolved in: #90

Can you please review and let me know?

@arran4 arran4 self-assigned this Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants