Disable Database Introspection on Startup #186
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Disables automatic database introspection on
Database
startup.A simple collection name check is now performed by default, comparing available db schemas with whatever, if anything, is already available in
DeclaredSchemas
, storing foreign schema collection names in the adapter for later use.DeclaredSchemas
is now aware of schemas' collection names.Any Conduit module schemas attempting to be registered using a used collection name now get their actual collection name prefixed by
cnd_
.Introspection can still be triggered through a
POST
@/admin/database/introspection
.Added a new admin route (
GET
@/admin/database/introspection
) for retrieving information on current introspection state:Added
getPendingSchema
(by id) admin route as requested by UI team.Fixed
finalizeSchemas
admin route not throwing on empty schemas array input.Note: Building this after updating node_modules post #184 depends on #185 getting merged as node build types have been messed up.
What kind of change does this PR introduce?
Does this PR introduce a breaking change?
The PR fulfills these requirements:
main
branchfix #xxx
, where "xxx" is the issue number)If adding a new feature, the PR's description includes: