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.

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

Add django-admin startapp template #3358

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

dalrrard
Copy link
Contributor

Description

django-admin has a --template flag that lets you specify a directory to
base new apps off of. I've added a startapp_template directory to the
top level directory, updated the post gen hook to remove any files that
may not be needed, and updated the documentation for this new process.

Checklist:

  • I've made sure that tests/test_cookiecutter_generation.py is updated accordingly (especially if adding or updating a template option)
  • I've updated the documentation or confirm that my change doesn't require any updates

Rationale

This lets people semi-automatically create new apps without doing a lot of manual file moving and editing.

Fix #1876
Fix #1725
Fix #1505
Fix #881

django-admin has a --template flag that lets you specify a directory to
base new apps off of. I've added a `startapp_template` directory to the
top level directory, updated the post gen hook to remove any files that
may not be needed, and updated the documentation for this new process.
@Manjit2003
Copy link
Contributor

Please approve this :(

@browniebroke
Copy link
Member

I'm a bit on the edge with this: while it's solving the problem, I find it a bit odd to have an new app template in the repo. After all, the startapp command accepts an URL as template. But maybe having it in the repo means that it can evolve with each person taste...

# Conflicts:
#	docs/troubleshooting.rst
#	hooks/post_gen_project.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants