Major refactor of tutorial folder structure.

morrieinmaas commented 3 years ago

In accordance with isseue 61 description on GitHub

TODOs:

Fix all broken links within the notebooks
extend notebooks to cover mediation and multitenant
possibly introduce helper functions that cn be imported from extra folder
!!! If medation and multitenant live in folder 1. the docker-compose.yaml and manage script need to be adjusted to accomodate for that setup. TBD what that looks like. More info here

Description

see issue 61

Checklist

[ ] I have followed the Contribution Guidelines and Code of Conduct
[ ] I have commented my code following the OpenMined Styleguide
[ ] I have labeled this PR with the relevant Type labels
[ ] My changes are covered by tests

wip-abramson commented 3 years ago

Sorry to be a pain but wondering about a slight refactor of tutorial structure for the first tutorial.

Basically Proofs tutorials wont work unless they have first issued themselves a credential. So perhaps it makes sense just to condense into the Credentials folder?

I also think Out of Band tutorial is an Advanced concept not a messaging one.

Like the tutorial names though!

morrieinmaas commented 3 years ago

@wip-abramson I'll refactor like that. Makes sense to me.

morrieinmaas commented 3 years ago

@wip-abramson @lohanspies Please have a look. I'm happy for any feedback. Lesson learned, pulled too much into one task. that got a bit out of hand volume-wise.

lohanspies commented 3 years ago

at got a bit out of hand volume-wise.

Agree. Let us keep it focussed on small achievable chunks.

OpenMined / PyDentity

Major refactor of tutorial folder structure. #70

Description

Checklist