Skip to content

[Issue 18] Added django chapter model#32

Open
zephris wants to merge 16 commits into
mainfrom
issue-18-Create_Chapter_Model
Open

[Issue 18] Added django chapter model#32
zephris wants to merge 16 commits into
mainfrom
issue-18-Create_Chapter_Model

Conversation

@zephris

@zephris zephris commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Change Summary

Added string function to fix object name setting in admin panel

Change Form

Fill this up (NA if not available). If a certain criteria is not met, can you please give a reason.

  • The pull request title has an issue number
  • The change works by "Smoke testing" or quick testing
  • The change has tests
  • The change has documentation

Other Information

Related issue

@zephris zephris self-assigned this Jun 24, 2026
@zephris zephris requested a review from Games4Doritos June 24, 2026 10:28
@zephris zephris added the backend Task must have a back end issue label Jun 24, 2026
@zephris zephris removed the request for review from Games4Doritos June 24, 2026 10:44

@Games4Doritos Games4Doritos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, I just have a very minor change: explicitly return str(self.name) to guarantee no type errors. May not even have an effect but will make the code look more correct I guess

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

Labels

backend Task must have a back end issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create Chapter Model

2 participants