-
Notifications
You must be signed in to change notification settings - Fork 6
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
Aggrid #74
Conversation
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.
@hash-bash check comments, and we merge soon
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.
shall we rm
this file?
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.
Yes, it doesn't serve any purpose, I kept it for later but we can create new one whenever required.
parkour_app/wui/settings/base.py
Outdated
# CORS Configuration | ||
CORS_ALLOW_CREDENTIALS = True | ||
CORS_ALLOW_ALL_ORIGINS = False | ||
CORS_ALLOWED_ORIGINS = ['http://localhost:5173'] |
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.
maybe we should move to parkour.env? if we still need it after bundling, gotta check what django-vite plugin offers (it may be that CORS headers are dealt with by that extension?)
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.
Sure, lets check about it and take steps accordingly while containerization.
docker-compose.yml
Outdated
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.
run pre-commit install
pls
No description provided.