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.

Sign up for GitHub

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

Jump to bottom

Remove permalink field from models #84

Open
stuartmaxwell opened this issue Nov 27, 2024 · 1 comment
Open

Remove permalink field from models #84

stuartmaxwell opened this issue Nov 27, 2024 · 1 comment
Assignees
@stuartmaxwell
Labels
refactor Refactor existing code

Comments

@stuartmaxwell
Copy link
Owner

We now have the url field, and we have url_utils.py for the central logic for figuring out URL paths. I think permalink is now redundant.
@stuartmaxwell stuartmaxwell added the design decision Needs consideration and design label Nov 27, 2024
@stuartmaxwell stuartmaxwell self-assigned this Nov 27, 2024
@stuartmaxwell stuartmaxwell moved this to Ready in DJ Press Nov 28, 2024
@stuartmaxwell stuartmaxwell moved this from Ready to In progress in DJ Press Nov 28, 2024
@stuartmaxwell stuartmaxwell mentioned this issue Dec 26, 2024
Merged
@stuartmaxwell stuartmaxwell moved this from In progress to Done in DJ Press Dec 26, 2024
@stuartmaxwell stuartmaxwell closed this as completed by moving to Done in DJ Press Dec 26, 2024
@stuartmaxwell
Copy link
Owner Author

We removed it from the Post model, but it can be removed from Category as well. See the Tag implementation for details.

@stuartmaxwell stuartmaxwell reopened this Jan 23, 2025
@stuartmaxwell stuartmaxwell moved this from Done to Backlog in DJ Press Jan 23, 2025
@stuartmaxwell stuartmaxwell added refactor Refactor existing code and removed design decision Needs consideration and design labels Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@stuartmaxwell stuartmaxwell

Labels
refactor Refactor existing code
Projects
DJ Press
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
1 participant
@stuartmaxwell