-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
App/exchange v2 #2493
App/exchange v2 #2493
Conversation
This pull request is being automatically deployed with Vercel (learn more). docs – ./apps/docs🔍 Inspect: https://vercel.com/cal/docs/9Z9teNh6qos31USF7NnZCNNdp6os [Deployment for e7035cd canceled] calendso – ./apps/web🔍 Inspect: https://vercel.com/cal/calendso/JDSxTav9UWCpxnYqhqb81DcHe8RS [Deployment for e7035cd failed] |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@zomars do you need reviews? |
@zomars do we need to add the app's data to |
@joeauyeung @PeerRich This PR is a little stale. Will re-draft until is properly updated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job @joeauyeung. Left a few comments to tackle 😉
defaultValues: { | ||
username: "", | ||
password: "", | ||
url: process.env.EXCHANGE_DEFAULT_EWS_URL || "", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@joeauyeung is this env documented somewhere? EXCHANGE_DEFAULT_EWS_URL
Reopens #2376
What does this PR do?
Adds Microsoft Exchange 2013 and 2016 to the app store.
This branch is created from
main
rather than the oldmonorepo/app-store
Fixes #2141
Type of change
How should this be tested?