docs: Add getting started guide for email authentication setup#560
docs: Add getting started guide for email authentication setup#560Zfinix wants to merge 2 commits into
Conversation
…ng setup steps and improving readability
Swiftaxe
left a comment
Swiftaxe
left a comment
There was a problem hiding this comment.
Looks clean! Added a few comments.
| @@ -0,0 +1,92 @@ | |||
| # Setup | |||
There was a problem hiding this comment.
Consider a more descriptive title. Setup was also used the old page, which keeps it as a filename, which is a bit conflicting.
| - Docker installed and running, so the server can start its database. | ||
| - The Flutter SDK installed, so you can run the app. | ||
|
|
||
| ## What you get out of the box |
There was a problem hiding this comment.
The whole "What you get out of the box" is a filler not needed. It repeats what is already said at the top. Remove it.
|
|
||
| ```bash | ||
| cd my_project_server | ||
| docker compose up --build --detach |
There was a problem hiding this comment.
Docker and dart run is not relevant for 3.5. Update to conform to latest version.
|
|
||
| ```bash | ||
| cd my_project_flutter | ||
| flutter run |
There was a problem hiding this comment.
No Flutter run needed in latest version.
| @@ -1,6 +1,10 @@ | |||
| # Setup | |||
| # Migrate to the new auth module | |||
There was a problem hiding this comment.
If this will be the migration guide, we need to fully test it as that and take the other attempts at migration guides into consideration (ie the old PR still hanging)
2 participants
No description provided.