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

Add DTSTAMP support #30

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

tcitworld
Copy link
Contributor

It's a required attribute

@tcitworld tcitworld force-pushed the feature/add-dt-stamp-support branch from 3526bb4 to e16a3a0 Compare January 15, 2021 08:44
@tcitworld tcitworld force-pushed the feature/add-dt-stamp-support branch from 26a2645 to 870b3e1 Compare April 10, 2022 16:56
It's a required attribute

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
@tcitworld tcitworld force-pushed the feature/add-dt-stamp-support branch from 870b3e1 to 222fce9 Compare April 10, 2022 17:18
@ibooking-sigurd
Copy link

Hi, are there any plans/chances for merging of this PR and a new version release in the near future?

@dfalling
Copy link

dfalling commented Sep 9, 2023

Hi, are there any plans/chances for merging of this PR and a new version release in the near future?

This repo's in maintenance mode. I switched to this fork:
https://github.com/tcitworld/icalendar.git

Mix.exs:
{:icalendar, git: "https://github.com/tcitworld/icalendar.git"},

@odarriba
Copy link

Hello,

As this is a required attribute, wouldn't it fall in the "bug fixes" category?
We are seeing clients are rejecting icals created by this library because of this and it was on the RFC since its creation, so it is not really a "new feature" :D

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.

6 participants