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
nv-3687-consolidate-provider-npm-packages-into-a-single #5453
nv-3687-consolidate-provider-npm-packages-into-a-single #5453
Conversation
27a1b72
to
96ce602
Compare
✅ Deploy Preview for dev-web-novu ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for novu-design ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
✂️
8727f0f
to
136a7d8
Compare
62166a1
to
07abeeb
Compare
Need to make sure we also update our docs for community: https://docs.novu.co/community/add-a-new-provider |
bf8446d
to
105404c
Compare
7f53872
to
8007dfd
Compare
.cspell.json
Outdated
@@ -731,5 +733,6 @@ | |||
".env.local", | |||
".env.production", | |||
".env.test", | |||
"./*" |
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.
I guess this one is by mistake?
.gitmodules
Outdated
[submodule "enterprise"] | ||
path = .source | ||
url = [email protected]:novuhq/packages-enterprise.git |
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.
Why did we remove this one?
@@ -1,6 +1,6 @@ | |||
packages: | |||
# all packages in subdirs of packages/ and components/ | |||
- 'providers/*' | |||
- 'automation' |
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.
I believe this is also not needed anymore right?
2557b3b
to
8b4d499
Compare
This reverts commit f566222.
What changed? Why was the change needed?
https://linear.app/novu/issue/NV-3687/consolidate-provider-npm-packages-into-a-single-novuproviders-package
https://novu.slack.com/archives/C06V6G9ERFZ
The Chance concluded of multiple parts, they are seperated by commits to make the review easier as this is a Very Big PR
Changes needed to be made to imports, changing '@novu/{providerName} --> '@novu/providers.
Packege.json dependecies needed to be changed to accomedate the new strature.
Screencast of the feature
https://www.loom.com/share/4e932a857ab84a80b211eab01a2373a5
Expand for optional sections
Related enterprise PR
Special notes for your reviewer