-
-
Notifications
You must be signed in to change notification settings - Fork 336
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
Remove database keys API #313
Comments
@kgodey Thanks. I will start work on this one then. I will start setting up my code. |
Thanks @arvind-1! Please feel free to join us on Matrix and ask questions either there or on the issue. |
@kgodey i am facing following error while trying run app. Could you help me on this?
|
@arvind-1 Did you run all the local development steps in the README: https://github.com/centerofci/mathesar#local-development? It looks like you may not have run the |
@kgodey I haven't run that. I tried right after docker-compose up. Now that i followed other two steps it works alright now. Thanks. |
@kgodey I went through all the change that have made for this. To be clear here i have to delete file |
@arvind-1 That is not correct. You have to delete all the code related to serving the |
Problem
The database keys API is deprecated, we don't need it around.
Proposed solution
Delete code associated with it once the frontend switches away from using it.
Additional context
The text was updated successfully, but these errors were encountered: