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

Quickstart django astro #2443

Merged
merged 38 commits into from
Aug 30, 2023
Merged

Conversation

sixhobbits
Copy link
Collaborator

@sixhobbits sixhobbits commented Aug 11, 2023

Depends on https://github.com/ritza-co/fusionauth-quickstart-python-django-web being added to fusionauth org

TODO - still needs a github action to test [edit done ✅]


@sixhobbits sixhobbits requested a review from mooreds August 11, 2023 13:27
@mooreds mooreds added the content Alert marketing to this change label Aug 15, 2023
Copy link
Contributor

@mooreds mooreds left a comment

Choose a reason for hiding this comment

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

  • I saw a we or two, please use you.

  • Why do we need CORS open to port 5000 in the example app in the kickstart?

  • remove the django guide example from the example apps data file (`site/_data/exampleapps.yml)

  • add a task to archive the guide example to the GH PR (I can do it, just want a task in the PR desc)

  • add a redirect to the cloudfront redirector (since this was previously a doc tutorial)

edit src/cloudfront/fusionauth-website-request-handler.js and look for the rails quickstart line:

rd[d+'/tutorials/integrate-ruby-rails']= '/docs/quickstarts/quickstart-ruby-rails-web';

add a similar one for the python django quickstart


Now create a `requirements.txt` file to list the project dependencies. (All the files in this guide already exist in the `complete-application` folder if you want to copy them.)

`requirements.txt`
Copy link
Contributor

Choose a reason for hiding this comment

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

This looks a bit weird. Can you say something like:

Suggested change
`requirements.txt`
Here's what `requirements.txt` should look like:


Now create the `.env` file in your `mysite` folder and add the following to it (note that this is a different environment file to the one in the root folder used by Docker for FusionAuth).

`.env`
Copy link
Contributor

Choose a reason for hiding this comment

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

Same as above, a bit weird to have the file name hanging out.

Open a new terminal. It is easier to run commands from within your `mysite` directory, so `cd` into it.

```shell
source venv/bin/activate;
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's not use any ; on the shell commands, please.


Overwrite your `mysite/settings.py` contents with the following code.

`settings.py`
Copy link
Contributor

Choose a reason for hiding this comment

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

Same issue here with the filename hanging out without any context.


Overwrite the `mysite/urls.py` file with the following code.

`urls.py`
Copy link
Contributor

Choose a reason for hiding this comment

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

Same issue here with the filename hanging out without any context.

* Copy the `static` folder with its contents from `complete-application/mysite/app/static` to your `/mysite/app/static` location.
* Copy the `templates` folder with its contents from `complete-application/mysite/app/templates` to your `/mysite/app/templates` location.

There are only two lines in these templates related to authentication. In the home template, there is a login button.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
There are only two lines in these templates related to authentication. In the home template, there is a login button.
There are only two lines in these templates related to authentication. In the `home.html` template, there is a login button.

Update the default SQLite database before testing.

```bash
python manage.py migrate
Copy link
Contributor

Choose a reason for hiding this comment

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

where do I need to be to run manage.py? I guess we didn't tell folks to cd anywhere, so I'm still supposed to be fusionauth-quickstart-python-django-web/mysite.

If the app is not running, you can use the following command.

```bash
source venv/bin/activate && python mysite/manage.py runserver
Copy link
Contributor

Choose a reason for hiding this comment

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

Would I do this in a new window? Why do I need the venv command? Where should I run this from?

Didn't I start the server up above and you haven't told me to stop it? I'm confused.

Copy link
Contributor

Choose a reason for hiding this comment

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

Should remove this file entirely.

@sixhobbits sixhobbits requested a review from mooreds August 22, 2023 08:23
Copy link
Contributor

@mooreds mooreds left a comment

Choose a reason for hiding this comment

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

2 minor changes. Need to put this through SEO review, but other than that, this is good to go.

Copy link
Contributor

@bradmccarty bradmccarty left a comment

Choose a reason for hiding this comment

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

@mooreds My only suggestion is RE: the title. Left a comment inline.

@mooreds mooreds merged commit 82b652e into FusionAuth:master Aug 30, 2023
alex-fusionauth pushed a commit that referenced this pull request Sep 5, 2023
* Added astro django menu entry for article

* Correct Rails to Flask typo

* Change references to new quickstart doc everywhere

* Work on python doc

* Getting old django quickstart to work

* Old django quickstart now works complete in astro with code. Now to make it meet the new standard

* English

* English

* Finished draft

* text updates

* text updates

* Python Django Quickstart language edit

* Python Django Quickstart proofread

* Update astro/src/content/quickstarts/quickstart-python-django-web.mdx

Co-authored-by: Dan Moore <[email protected]>

* Change we to you

* remove the django guide example from the example apps data file

* Add rediretion for new guide

* Remove filename above code snippets

* Remove ; from shell commands

* Change absolute links to relative

* Name file

* Remove integrate-python-django.adoc

* Change from title case to start case

* Tell Dan we don't need to cd at any point

* Explain why we are using venv

* Remove unnecessary command to start Django server

* Clarified all commands in directories

* Comments on windows

* Add warning about passwords in git

* fix relative urls

* Revised Python Django Quickstart language edit

* Apply suggestions from code review

Co-authored-by: Dan Moore <[email protected]>

* update Elastic to Elasticsearch

---------

Co-authored-by: RichardJECooke <[email protected]>
Co-authored-by: Tatenda <[email protected]>
Co-authored-by: worktheclock <[email protected]>
Co-authored-by: Dan Moore <[email protected]>
alex-fusionauth added a commit that referenced this pull request Sep 22, 2023
* initial install

* update

* pr fixes

* remove type file

* pr updates

* remove photo refs

* Revert "disable basic auth for testing (#2442)"

This reverts commit f34339f.

* Revert "disable basic auth for testing (#2442)"

This reverts commit f34339f.

* add opengraph image

* reauth dev

* don't limit header image

* article to education

* bradm publishes inc5000 blog post (#2452)

* bradm publishes inc5000 blog post

* fixed an H2

* Fixed the absolute URLs

---------

Co-authored-by: Brad McCarty <[email protected]>

* fixing the inc5000 placement (#2454)

Co-authored-by: Brad McCarty <[email protected]>

* removing extra / at end of link (#2455)

* 1.47 release announcement didn't get correct category, prob due to when it was published

* add in supabase (#2456)

* spellcheck astro site too (#2457)

* removed extra space from URL

* tbd for spellcheck

* updated with more known words

* uninstall compress so spellcheck runs faster

* Migrated a missed post, couple of fixups

* pre-compressing images

* better blurbage

* Header and footer changes

* trailing slash on blog pages

* oops

* oops

* oops...

* fix for latest

* fix extra space (#2458)

* fixing extra space

* Mooreds/update spellcheck for astro (#2459)

* make sure we catch all astro files for spellcheck

* Make sure get all astro html, and exclude the ESC button on devtools

Note that this currently picks up a lot of truncated words, so don't fix this until the blurbing works better

* Add fusiondesk post (#2380)

Adds a react SDK example blog post.

---------

Co-authored-by: Colin Frick <[email protected]>

* No trailing slash for blog items. (#2460)

This is based on this PR: withastro/astro#6453 which landed a few months ago.

* Mooreds/update content check (#2462)

* updated content check to look for categories and check titles

* fix category of this post

* Mooreds/fix cats blog post (#2463)

* corrected tags

* corrected category

* commented out title check until Brad decides

* don't worry about compress right now (#2464)

* Bradm/vc free (#2465)

* pushing this up for Lyle to fix

* small updates

* brad publishes VC Free post

* fixed category failure

---------

Co-authored-by: Brad McCarty <[email protected]>

* Mooreds/fix spelling aug 17 2023 (#2466)

* added more known words

* corrected spelling errors or typos

* Mooreds/check for absolute urls (#2467)

* check for absolute fusionauth.io urls and empty markdown links `[text...]()`.

we want to use relative urls

* Mooreds/1951 upgrade guide (#2461)

Upgrade guide overhaul

* Mooreds/add color about downtime (#2468)

* added note about downtime expectations

* Revise docsdevreadme for new world (#2471)

* Revise docsdevreadme for new world

* emphasized header stuff

* added note about start case

* trying to fix linkcheck (#2472)

we get a lot of connection failure errors

* added note about links

* Mooreds/fix 404s from linkcheck (#2473)

* fixing images

* handling another 404 image

* handling 404 images

* Image URL wasn't correctly updated

* fixed unmoved image

* fixed busted link

* correct sitemap submission params

* only run 1/day

* Mooreds/fix spelling aug 18 2023 (#2474)

* ignore tag files

* fixed typos

* more known words

* Docs & dev readme style grammar fix (#2475)

Co-authored-by: worktheclock <[email protected]>

* add id for article (#2469)

* clean out the s3 bucket on deploy

* fix markdown that made it into asciidoc (#2476)

* Expanded 2xl dimension on index page for sidebar

* Updated nextjs header image

* Update index footer blurb text

* clean out the s3 bucket on deploy

* Document options for change password during a slow migration (#2477)

* Capturing some knowledge that came up during a support discussion

* minor formatting change

* exclude stuff that isn't "content" from the search

* oops

* debounce the search and only update the dom when done

* fetch faster

* fix merge mishap

* fix busted image (#2479)

* add 504 response code for /api/two-factor/login (#2480)

* clarifying doc (#2481)

* fixed breaking external links (#2482)

* fixed breaking external links

* fixed breaking external tag links

* redirect old category names

* fix busted link

* tagging additional conference posts (#2483)

* add note about CLI goals (#2484)

* brad publishes mark's unlimited domains post (#2485)

Co-authored-by: Brad McCarty <[email protected]>

* updated unlimited custom domain header style

* Updated diagram img path

* updated unlimited domain diagram

* update to img rounded corners

* fix the blog blog link

* remove nbsps as they did not look on-purpose

* Add notes about loginId

* Blog width update

* Fixing the changebank link (#2487)

Want to send folks to the right parody video

* Update sitemap.xml

Compare Auth0 page going live on brochure site.

* fix changebank logo, added multiplattform font definitions. (#2489)

* Added link to quickstarts (#2490)

* Make it clearer which webhook types are for premium, enterprise and community plans (#2446)

* Add treefort interview (#2414)

* initial import

* spell check

* rename

* updating header image

* shrinking image

* fixing name mispelling

* changing title per SEO review

* prepping treefort interview

* changing title

* added some links

* fixed tags and categories

* Use new image

* fixed post date, incorrect tag

* Quickstart django astro (#2443)

* Added astro django menu entry for article

* Correct Rails to Flask typo

* Change references to new quickstart doc everywhere

* Work on python doc

* Getting old django quickstart to work

* Old django quickstart now works complete in astro with code. Now to make it meet the new standard

* English

* English

* Finished draft

* text updates

* text updates

* Python Django Quickstart language edit

* Python Django Quickstart proofread

* Update astro/src/content/quickstarts/quickstart-python-django-web.mdx

Co-authored-by: Dan Moore <[email protected]>

* Change we to you

* remove the django guide example from the example apps data file

* Add rediretion for new guide

* Remove filename above code snippets

* Remove ; from shell commands

* Change absolute links to relative

* Name file

* Remove integrate-python-django.adoc

* Change from title case to start case

* Tell Dan we don't need to cd at any point

* Explain why we are using venv

* Remove unnecessary command to start Django server

* Clarified all commands in directories

* Comments on windows

* Add warning about passwords in git

* fix relative urls

* Revised Python Django Quickstart language edit

* Apply suggestions from code review

Co-authored-by: Dan Moore <[email protected]>

* update Elastic to Elasticsearch

---------

Co-authored-by: RichardJECooke <[email protected]>
Co-authored-by: Tatenda <[email protected]>
Co-authored-by: worktheclock <[email protected]>
Co-authored-by: Dan Moore <[email protected]>

* bradm blog SEO updates (#2496)

Co-authored-by: Brad McCarty <[email protected]>

* adding color to google custom parameters section (#2497)

* fix busted include (#2498)

* add in angular sdk (#2499)

* SCIM server doc updates (#2501)

Added more details, fixed some inconsistencies.

* add 504 webhook login response code to /api/login, /api/passwordless/login and /api/identity-provider/login endpoint docs (#2495)

* Mcr/blog get more value (#2312)

* get more value post

* updating link to sample application

* updating with changes from Brad

* Update site/_posts/2023-07-11-get-more-value-out-of-fusionauth.md

reformat paragraph

Co-authored-by: Dan Moore <[email protected]>

* update links verbiage

Co-authored-by: Dan Moore <[email protected]>

* Update conclusion verbiage

Co-authored-by: Dan Moore <[email protected]>

* Update Link verbiage

Co-authored-by: Dan Moore <[email protected]>

* making changes from review

* finishing merge

* small clean ups

* making changes from review

* update word

* Update site/_posts/2023-07-11-get-more-value-out-of-fusionauth.md

update Burger sp

Co-authored-by: Dan Moore <[email protected]>

* Update site/_posts/2023-07-11-get-more-value-out-of-fusionauth.md

updating language

Co-authored-by: Dan Moore <[email protected]>

* Update site/_posts/2023-07-11-get-more-value-out-of-fusionauth.md

Update Language

Co-authored-by: Dan Moore <[email protected]>

* updating typos

* adding to example-apps

* removing Net language and capatilizing blog post headers.

* moving to astro

* tweaking for display

* update categories

* updating date and image

* removing old images

---------

Co-authored-by: Dan Moore <[email protected]>

* fixing tag that was preventing build (#2504)

* Update image for Value

* updates based on Dan's PR Review

* add makechange

* correct merge conflict

* update makechange

* add Login Link

---------

Co-authored-by: Andy Pai <[email protected]>
Co-authored-by: Lyle Schemmerling <[email protected]>
Co-authored-by: Brad McCarty <[email protected]>
Co-authored-by: Brad McCarty <[email protected]>
Co-authored-by: Dan Moore <[email protected]>
Co-authored-by: Colin Frick <[email protected]>
Co-authored-by: Gareth Dwyer <[email protected]>
Co-authored-by: worktheclock <[email protected]>
Co-authored-by: Sean Bryant <[email protected]>
Co-authored-by: Brent Halsey <[email protected]>
Co-authored-by: Daniel DeGroff <[email protected]>
Co-authored-by: Sean Bobby <[email protected]>
Co-authored-by: Michael Roth <[email protected]>
Co-authored-by: RichardJECooke <[email protected]>
Co-authored-by: Tatenda <[email protected]>
Co-authored-by: Mark Robustelli <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Alert marketing to this change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants