Skip to content

Dev#19

Merged
ngovinh2k2 merged 2 commits into
mainfrom
dev
Apr 17, 2026
Merged

Dev#19
ngovinh2k2 merged 2 commits into
mainfrom
dev

Conversation

@ngovinh2k2
Copy link
Copy Markdown
Member

What?

Why?

How?

Testing?

  • Functional Testing
  • Security
  • Performance
  • Error Handling
  • Code Quality
  • Documentation
  • Database
  • Deployment
  • Final Review

Anything Else?

@ngovinh2k2 ngovinh2k2 merged commit d3996f2 into main Apr 17, 2026
1 check passed
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the organization view to include a template field in the API response. A review comment identifies a critical issue where the Organization model lacks this field, which will cause an AttributeError at runtime.


return Response({"result": result}, status=status.HTTP_200_OK)
return Response(
{"result": result, "template": organization.template},
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

The Organization model does not have a template field defined in apps/organization/models.py. Accessing organization.template will cause an AttributeError at runtime. Please verify if this field needs to be added to the model or if a different attribute should be used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant