Skip to content
This repository was archived by the owner on Jun 9, 2025. It is now read-only.

Conversation

@ArnyminerZ
Copy link
Member

Signed-off-by: Arnau Mora arnyminer.z@gmail.com

Closes #81

Signed-off-by: Arnau Mora <arnyminer.z@gmail.com>
@ArnyminerZ ArnyminerZ self-assigned this Feb 8, 2023
@ArnyminerZ ArnyminerZ linked an issue Feb 8, 2023 that may be closed by this pull request
3 tasks
Signed-off-by: Arnau Mora <arnyminer.z@gmail.com>
Signed-off-by: Arnau Mora <arnyminer.z@gmail.com>
@ArnyminerZ ArnyminerZ marked this pull request as ready for review March 20, 2023 17:46
@ArnyminerZ ArnyminerZ mentioned this pull request Mar 20, 2023
3 tasks
Signed-off-by: Arnau Mora <arnyminer.z@gmail.com>
@ArnyminerZ ArnyminerZ requested a review from rfc2822 March 20, 2023 20:32
Copy link
Member

@rfc2822 rfc2822 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, nice to finally have this. I have added some minor comments.

Can you please try whether these cases (see original issue bitfireAT/davx5#165) are covered by the fix too? Probably we don't need separate tests for them because these tests would belong to ical4j and are not that important for us (I'd just like to know whether these cases are covered too):

GEO:
GEO:48.786829
GEO:48.786829;27.801019

Signed-off-by: Arnau Mora <arnyminer.z@gmail.com>
@ArnyminerZ
Copy link
Member Author

Can you please try whether these cases (see original issue https://github.com/bitfireAT/davx5/issues/165) are covered by the fix too?

I've manually tested it by replacing the GEO: tag on invalid-geo.ics and it works correctly.

ArnyminerZ and others added 3 commits March 23, 2023 14:46
Signed-off-by: Arnau Mora <arnyminer.z@gmail.com>
Signed-off-by: Arnau Mora <arnyminer.z@gmail.com>
@rfc2822 rfc2822 force-pushed the 81-ignore-invalid-properties branch from e7ec556 to 9013a92 Compare March 23, 2023 15:39
@rfc2822 rfc2822 merged commit 8c52fc8 into main Mar 23, 2023
@rfc2822 rfc2822 deleted the 81-ignore-invalid-properties branch March 23, 2023 15:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ignore invalid properties

2 participants