Remove configFile URL param support #724
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.
Description
Removes support for an obscure and undocumented way of loading an initial connection.
To use the feature a user must:
http://localhost:5173/?configFile=connection.json
If those steps are followed then the database will be initialized with the connection data inside the file.
However, if any connections already exist this logic will be skipped. So it is only useful for setting up a brand new user of Graph Explorer.
This process is undocumented and is tedious to set up. There are better approaches to configuring initial connections for new users. So my theory is that this feature was initially created for the original developers to test out new browsers or to quickly setup their test environment after wiping their local database.
Validation
Related Issues
Check List
license.
pnpm checks
to ensure code compiles and meets standards.pnpm test
to check if all tests are passing.Changelog.md
.