Skip to content
This repository was archived by the owner on Apr 16, 2023. It is now read-only.

Conversation

@meshy
Copy link
Owner

@meshy meshy commented Jul 27, 2015

Depends on #44

I didn't do this TDD, so it needs tests. Will have to add them retroactively.

I'm starting to doubt if django-sirtrevor is really the right thing for the job. SirtrevorJS seems good, but I don't like that this forces one to add urls for it.

Copy link
Owner Author

Choose a reason for hiding this comment

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

I think I missed the migration for this, and it needs validation in the admin form.

The slug is required unless there's no parent (no parent and slug makes it the root). There can only be one root.

@meshy
Copy link
Owner Author

meshy commented Aug 2, 2015

I hope to get this tested and merged before tackling the other issues that have arisen as a result of it (#62 #61)

Copy link
Owner Author

Choose a reason for hiding this comment

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

@register(Route)

@meshy
Copy link
Owner Author

meshy commented Jan 5, 2016

Blocked by #71

@meshy
Copy link
Owner Author

meshy commented Feb 7, 2016

Right, I think this avenue is a write-off for the moment. Closing.

@meshy meshy closed this Feb 7, 2016
@meshy meshy deleted the django-admin branch February 7, 2016 15:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants