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

Adding option for creating template of Positron (Django or Static) #69

Closed
wants to merge 9 commits into from

Conversation

Cheukting
Copy link

@Cheukting Cheukting commented May 22, 2023

Added option to create a project using Django or Static Position (supported by Toga). I will be using it for my tutorial but I don't know if it is useful to push back upstream - feel free to have a look and discuss.

PR Checklist:

  • All new features have been tested
  • All new features have been documented
  • I have read the CONTRIBUTING.md file
  • I will abide by the code of conduct

Happy to add tests and docs if it is desired upstream, let's talk about it.

Copy link
Member

@freakboy3742 freakboy3742 left a comment

Choose a reason for hiding this comment

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

One comment inline about the specifics of this template; however, I think there's a bigger design issue to tackle. See beeware/briefcase#392 for details.

# See https://docs.djangoproject.com/en/4.2/howto/deployment/checklist/

# SECURITY WARNING: keep the secret key used in production secret!
SECRET_KEY = 'django-insecure-86kqbg4s&53!bmq5)pd0hg@a%7b1pn_7&yw(tljy=xh12*!_u$' # noqa: E501
Copy link
Member

Choose a reason for hiding this comment

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

This should probably be generated by the template, rather than hard-coded with a warning.

Copy link
Author

Choose a reason for hiding this comment

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

we can also strip back the code to not include those generate by Django, anyway, will close this PR

@Cheukting
Copy link
Author

I am using this for the quick demo for now, will open a new PR when there is a solution for the GUI framework plug-in described in beeware/briefcase#1288

@Cheukting Cheukting closed this May 23, 2023
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.

2 participants