Comments (26)
I think that we will be submitting a PR to this project to try to add support for CardDAV, since none of the other open-source projects have MIT or MPL like licensing, as this one does.
Feel free! Once you do, I'll be sure to review and merge them in as soon as I can.
from caldav-adapter.
Thanks @sedenardi - if you have Matrix/Element, would love to have you join our chat. Either way, thanks for building this and releasing with MIT ❤️ 🙏 We're definitely planning to use and contribute @forwardemail.
https://forwardemail.net (click on Resources > Community > Matrix Chat)
from caldav-adapter.
Note for us, proId
is a typo, it should be prodId
from caldav-adapter.
Hi @titanism, I've added you as a collaborator to this repository. As I mentioned in a comment on another issue, I'm no longer actively developing this project personally, but I'm happy to grant you access to continue it.
I will say, this is my first experience with GitHub Collaborators, so bear with me and let me know if you don't have proper access to review and merge PRs. I can publish new releases to npm
when necessary, but we can discuss whether that's something you'd like access to do as well.
from caldav-adapter.
Thank you @sedenardi. I've accepted the invite on GitHub and will let you know if I have any issues.
I would love to have access to publish on npm
too, respecting SemVer. At @forwardemail we use np
to publish releases with detailed changelogs in the Releases tab of GitHub. See https://github.com/sindresorhus/np. You can grant us npm ownership by typing npm owner add titanism caldav-adapter
on CLI I believe.
from caldav-adapter.
Edit: The command is actually npm owner add titanism caldav-adapter
from caldav-adapter.
Done, let me know if there's anything else.
from caldav-adapter.
You're the best @sedenardi! We're adding a ton of tests and discovering how this all works, will keep you apprised. 🙏
from caldav-adapter.
It doesn't seem like we have write access still, are you sure you added the right user and it was successful @sedenardi ?
Error screenshot below:
![Screen Shot 2024-01-26 at 12 03 51 PM](https://private-user-images.githubusercontent.com/101466223/300069965-ff630ff2-5240-42b3-b73b-692776ff32a5.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjM3NjcwMzYsIm5iZiI6MTcyMzc2NjczNiwicGF0aCI6Ii8xMDE0NjYyMjMvMzAwMDY5OTY1LWZmNjMwZmYyLTUyNDAtNDJiMy1iNzNiLTY5Mjc3NmZmMzJhNS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwODE2JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDgxNlQwMDA1MzZaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT00NjBjZTA2NmFjNjRjNTFmNzQ0M2Q1NjgzZDgzN2M5YzdkYjVmOTk2NWFjOTUwZWIwYzMzNjM2MWQyNzg5MzA0JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.Gg70D7OxSm1NSa0LGzeBPdo-ZRu1mMzA9Wl61pRx6uU)
from caldav-adapter.
I ran npm owner add titanism caldav-adapter
and got a successful response.
npm notice INFO: User titanism invited to package caldav-adapter successfully.
+ titanism (caldav-adapter)
Is there an additional permissions I need to grant?
from caldav-adapter.
@sedenardi I got the email, it went to spam. Thank you for your help and patience. Separately, I've filed an issue with npm because their DKIM signature was not aligned since they use Amazon SES, and DMARC with "p=reject" ; therefore their email went to spam.
from caldav-adapter.
No worries, glad you got it sorted.
I've always found DMARC to be more of a footgun than helpful, so I'm not shocked to hear that.
from caldav-adapter.
We've found some core bugs in this package and did a complete rewrite.
For example, https://github.com/sedenardi/node-caldav-adapter/blob/4076fc67552ddbab2ead5dc52395b0b1497f2b35/src/common/tags.ts#L165-L168 should instead have resource
as the object property instead of response
. There were other typos like this similarly in the codebase, and we discovered these through tests.
Unless you are interested in transferring this repository to the @forwardemail organization @sedenardi, we plan to publish an independent fork of this project (with your original authorship retained) and an updated copyright to include us, under the same MIT license. Our fork will be under @forwardemail/dav
otherwise 🚀
Thank you so much for building this library, we've poured several hours into this and our fork is modernized.
from caldav-adapter.
I'm more than happy to transfer ownership!
For some context, this package was developed (and tested) for specific workflows and capabilities. I used other CalDAV implementations (Apple's, radicale, etc.) as well as the RFC spec as guidance, but concentrated on making sure the code paths I was using (often determined through testing against several clients) were correct. Any other code paths (like this example) were admittedly placeholders for if I ended up needing to implement them.
Please let me know what you need me to do to transfer the repo. I love that the countless hours I've poured into this are proving useful for others!
from caldav-adapter.
@sedenardi amazing 🙏 can you transfer it to @forwardemail organization on GitHub?
from caldav-adapter.
I received this error when I tried transferring it. Looks like you already have a fork, so not sure how you want to handle it.
![Screenshot 2024-02-02 at 14 25 47](https://private-user-images.githubusercontent.com/4225973/301950149-bafe3aaa-145c-47db-9842-d2ccc0513833.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjM3NjcwMzYsIm5iZiI6MTcyMzc2NjczNiwicGF0aCI6Ii80MjI1OTczLzMwMTk1MDE0OS1iYWZlM2FhYS0xNDVjLTQ3ZGItOTg0Mi1kMmNjYzA1MTM4MzMucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDgxNiUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA4MTZUMDAwNTM2WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9ODI4NzI5NGFjNjUwMjEwMDA4ZjVlZDEzNGIyZWFjNGY5NWYzMWViZjZlMDgxZGU1NDMzMTAyYjA5NDYwZDNkMCZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.qVd8idEDEcNN19UszHAl4k7XBb5u97hJSyvavVYy5Ng)
from caldav-adapter.
@sedenardi we just deleted our fork, try again? 😄
from caldav-adapter.
New error this time.
![Screenshot 2024-02-02 at 14 43 48](https://private-user-images.githubusercontent.com/4225973/301954364-9bb6358a-b35e-4a27-ae24-ff3ec96e8565.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjM3NjcwMzYsIm5iZiI6MTcyMzc2NjczNiwicGF0aCI6Ii80MjI1OTczLzMwMTk1NDM2NC05YmI2MzU4YS1iMzVlLTRhMjctYWUyNC1mZjNlYzk2ZTg1NjUucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDgxNiUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA4MTZUMDAwNTM2WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MTRhZDFkNTEyODkzMWZhYTc0MTMyNzA3ZjE2MzBlNTliY2M4MjllOGU0YjY1Y2I0M2VmNGZhMzE3MzU0ZDM2NyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.wvGdOKaoxEFnNd5EPT9dSMFpv1fpu3ICPj0SqJEEEFI)
from caldav-adapter.
sec
from caldav-adapter.
@sedenardi just invited you to the org, once you accept invite (check your inbox), should be good 🖖
from caldav-adapter.
Done!
from caldav-adapter.
@sedenardi Thank you 🙏 Will keep you as a maintainer 🖖
from caldav-adapter.
@sedenardi we made it happen! see forwardemail/forwardemail.net@57907d4 and https://github.com/forwardemail/caldav-adapter/releases/tag/v5.0.0
from caldav-adapter.
@titanism Congrats on getting your features out the door.
One thing that caught my eye is the removal of Typescript support in that release. While this library is now fully yours to do with what you want, I would strongly recommend reconsidering this decision for a 2 main reasons:
- Typescript (IMO) is now the de-facto standard among
npm
libraries. Removing TS typings may not only turn away folks from using it in their own applications (as it would for me if there were no types), but highly discourages outside developers from contributing. - While the application you're using this library in may not be Typescript, TS is backwards-compatible and does bring some nice type-hinting to most popular IDEs.
Feel free to reach out if you'd like to discuss this decision further. As I mentioned, the decision is fully in your hands now, but I mention it only to promote the broad use of and contribution to this open source library.
from caldav-adapter.
Hi there @sedenardi 👋
Yes we dropped TS - as we attempted to use it, but found too many issues with the previous code with TS, warnings, etc.
We also don't use TS for any of our packages (which have > 30M+ monthly downloads), and attempted to rewrite the code while still using TS, but came to the conclusion it's unnecessary complexity.
TypeScript and ESM are not the current standards (a majority > 60-70% of npm packages still use pure JS and CJS exports last time we checked). We are also maintainers of Express and Koa, and have found TS and ESM to be horrible experiences and huge time sinks.
Also see https://gist.github.com/joepie91/bca2fda868c1e8b2c2caf76af7dfcad3 and https://world.hey.com/dhh/turbo-8-is-dropping-typescript-70165c01.
from caldav-adapter.
Fair enough!
from caldav-adapter.
Related Issues (12)
- Which clients has this been tested with? HOT 2
- Adapt line folding to cover surrogate pairs HOT 4
- Webapp/Express framework support HOT 2
- CardDAV support HOT 2
- CI uses `actions` that use a `deprecated` version of `Node.js` HOT 1
- how to run? HOT 30
- ETag with Android HOT 4
- Allday events not working properly HOT 2
- Question: setting macOS HOT 1
- getEventsByDate seems to have not the correct response for iCal HOT 4
- xBuild multistatus handles empty responses incorrectly
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from caldav-adapter.