Skip to content

docs(tutorials): complete the storage step#71

Closed
nweldev wants to merge 7 commits intomainfrom
tuto-complete-storage
Closed

docs(tutorials): complete the storage step#71
nweldev wants to merge 7 commits intomainfrom
tuto-complete-storage

Conversation

@nweldev
Copy link

@nweldev nweldev commented Oct 12, 2022

@nweldev nweldev force-pushed the tuto-complete-storage branch from 1830f27 to b39334b Compare October 13, 2022 13:43
@nweldev nweldev requested a review from TheKinrar October 13, 2022 13:44
@nweldev
Copy link
Author

nweldev commented Oct 13, 2022

@TheKinrar: this step is now ready for review, but I'll keep it as a draft until we fix blindnet-io/product-management#983.

Just ignore the related warning.

Copy link
Contributor

@TheKinrar TheKinrar left a comment

Choose a reason for hiding this comment

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

Stumbled upon some typos, thought I would also report them, the result is this review is mostly about typos.

We'll have to make some changes to the tutorial(s) in like a week or two when we start using authentication, app scopes and the identity server, but I'd say this is fine for now!

@nweldev nweldev force-pushed the tuto-complete-storage branch from 29b449b to 325f565 Compare October 14, 2022 11:40
@nweldev
Copy link
Author

nweldev commented Oct 14, 2022

Thank you very much @TheKinrar. You are perfectly right. IMHO, the best option would be to split this tutorial into multiple component-specific ones and to replace it for real quick start either with a very simple one using the devkit monorepo once the docker-compose config is done to underline the on-premise approach, or a completely new SaaS approach.

@TheKinrar
Copy link
Contributor

I agree. Full self-hosting means at least four components, so we clearly need a docker-compose for that. Besides, it makes things easier, for us, and for users.

@TheKinrar TheKinrar self-assigned this Nov 16, 2022
@TheKinrar
Copy link
Contributor

Superseded by #90.

@TheKinrar TheKinrar closed this Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

docs(tutorials): complete the storage step

3 participants

Comments