dominic/localist
1
0
Fork 0
Code Issues 19 Pull requests 2 Projects 1 Releases Packages Wiki Activity Actions

added cors headers to settings file, and changed request.post to json.loads(request.body) for frontend #2

Merged
madeleinema merged 2 commits from updateviews into master 2026-06-20 17:59:53 -04:00
Conversation 0 Commits 2 Files changed 5 +23 -6
madeleinema commented 2026-06-20 17:51:23 -04:00
Collaborator
Copy link

I updated the

  • settings.py file to influce cors headers
  • requirements.txt to include cors-headers
  • Updated views/events, categories, and tags "request.post" to "json.loads(request.body)"
    Please approve.
I updated the - settings.py file to influce cors headers - requirements.txt to include cors-headers - Updated views/events, categories, and tags "request.post" to "json.loads(request.body)" Please approve.
madeleinema added 1 commit 2026-06-20 17:51:23 -04:00
dominic reviewed 2026-06-20 17:53:18 -04:00
api/views/category.py Outdated
@ -4,3 +4,3 @@
from api.views.base import BaseView
from web.models.category import Category
import json
dominic commented 2026-06-20 17:53:18 -04:00
Owner
Copy link

import json needs to be at the top of the file, in alphabetical order... there are three blocks of imports separated by blank space in between, first is python native packages, then third party packages, then packages we create.

import json

import django

import mydjango
import json needs to be at the top of the file, in alphabetical order... there are three blocks of imports separated by blank space in between, first is python native packages, then third party packages, then packages we create. ``` import json import django import mydjango ```
dominic commented 2026-06-20 17:53:44 -04:00
Owner
Copy link

fix in all files

fix in all files
dominic reviewed 2026-06-20 17:54:04 -04:00
localist/settings.py Outdated
@ -32,6 +32,7 @@ ALLOWED_HOSTS = []
# Application definition
INSTALLED_APPS = [
"corsheaders", # add before other apps
dominic commented 2026-06-20 17:54:04 -04:00
Owner
Copy link

remove ai comments, here and below

remove ai comments, here and below
dominic requested changes 2026-06-20 17:54:29 -04:00
Dismissed
dominic left a comment
Owner
Copy link

changes requested

changes requested
madeleinema added 1 commit 2026-06-20 17:58:07 -04:00
dominic approved these changes 2026-06-20 17:59:18 -04:00
madeleinema merged commit 175ffda130 into master 2026-06-20 17:59:53 -04:00
madeleinema deleted branch updateviews 2026-06-20 17:59:53 -04:00
Sign in to join this conversation.
Reviewers
No reviewers
dominic
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: dominic/localist#2
No description provided.
Delete branch "updateviews"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?