Skip to content
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

Give React SPA frontend specific things a home #904

Closed
wants to merge 6 commits into from

Conversation

hmpf
Copy link
Contributor

@hmpf hmpf commented Oct 9, 2024

Closes #900

We're moving away from the REACT frontend so we must make it easier to switch between the old and the new. One day we'll delete all the REACT frontend specific things, we ought to make it safe and easy.

This means settings-wrangling.

App settings come in two varieties:

  1. App specific settings, often with a prefix
  2. Standard django settings that the app assumes is set in a certain way

Much have been written about the best way to tackle 1. Not much about 2.

Furthermore standard django settings are of two types: Overridable (like DEBUG) and should be mutated in place (MIDDLEWARE, TEMPLATES..) The trend is towards more complex settings like the latter.

Finally, there's what to do about the root urls.py.

@hmpf hmpf self-assigned this Oct 9, 2024
@hmpf hmpf added frontend Has/needs companion issue in frontend API Affects Argus' REST API API v2 Ideas for API v2, backwards incompatible OK auth Touches the authentication/authorization subsystem labels Oct 9, 2024
@hmpf hmpf requested review from a team October 9, 2024 13:43
@hmpf hmpf added the help wanted Extra attention is needed label Oct 9, 2024
@elfjes
Copy link
Collaborator

elfjes commented Oct 9, 2024

Nice, I like where this is going :), Do you want to make spa an app?

@hmpf
Copy link
Contributor Author

hmpf commented Oct 11, 2024

Nice, I like where this is going :), Do you want to make spa an app?

It doesn't need to be. It has no models, templates, static files, template tags or other things that is autodiscovered.

The goal is to delete the directory and anything mentioning "spa" one happy day.

@hmpf
Copy link
Contributor Author

hmpf commented Oct 11, 2024

test_CI.py had DOS line ends which really sucks when "git add -p" with new code that has unix line ends. The commit "Strip \r from lineends" needs to go in .git_blame_ignore_revs when everything is ready.

Copy link

sonarcloud bot commented Oct 15, 2024

@hmpf
Copy link
Contributor Author

hmpf commented Oct 15, 2024

Replaced by #907

@hmpf hmpf closed this Oct 15, 2024
@hmpf hmpf deleted the spa-frontend-specific branch October 15, 2024 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API v2 Ideas for API v2, backwards incompatible OK API Affects Argus' REST API auth Touches the authentication/authorization subsystem frontend Has/needs companion issue in frontend help wanted Extra attention is needed
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Move everything depending on python-social-auth to separate sub-app
2 participants