-
-
Notifications
You must be signed in to change notification settings - Fork 244
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
Deprecate uniforms-bridge-graphql
package
#1322
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1322 +/- ##
==========================================
- Coverage 94.55% 94.47% -0.09%
==========================================
Files 231 229 -2
Lines 3823 3730 -93
Branches 1030 1002 -28
==========================================
- Hits 3615 3524 -91
+ Misses 82 81 -1
+ Partials 126 125 -1 ☔ View full report in Codecov by Sentry. |
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.
If we leave the docs about the GraphQL bridge, shouldn't we add a warning that the package has been deprecated? I suggest using admonition.
Deprecate the GraphQL bridge, as mentioned in this issue (#1315).
uniforms
repositorygraphql
anduniforms-bridge-graphql
packages frompackage.json
filesNote
I left the
GraphQLBridge
docs. We will keep that in the uniforms v3 docs.