PyDentity icon indicating copy to clipboard operation
PyDentity copied to clipboard

Refactor Project to Use the aries-cloudcontroller class

Open wip-abramson opened this issue 4 years ago • 2 comments

Description

The aries-basic-controller library has now been extracted into it's own repo and is being developed under DIDx (@lohanspies ). The aim is to turn this into production ready library with a complete CI/CD pipeline and release strategy etc. Repo is here -https://github.com/didx-xyz/aries-cloudcontroller-python

Currently the aries-basic-controller library still exists in PyDentity and all tutorials and projects are using this code as is. We should do a complete refactor of this, so all notebooks use the deployed https://pypi.org/project/aries-cloudcontroller/ package. This version should be fixed, as the aries-cloudcontroller is under active development and staying with the latest version could cause the tutorials to get out of sync.

It should be a relatively straightforward set of changes that will help tidy up the repo

wip-abramson avatar Apr 14 '21 13:04 wip-abramson

I've addressed this here by adding the did-xyz repo as a subtree.

in order to pull new changes from there one has to

git subtree pull --prefix libs/aries-basic-controller https://github.com/didx-xyz/aries-cloudcontroller-python.git main

it is also possible to push changes from here to the parent to a specific branch like so

git subtree push --prefix libs/aries-basic-controller https://github.com/didx-xyz/aries-cloudcontroller-python.git the_branch

TODOs:

  • Replace old references to aries_basic_controller with the new aries_cloudcontroller in the code
  • remove the old code that is currently under aries-basic-controller_old once the previous point has been addressed.

morrieinmaas avatar Apr 14 '21 19:04 morrieinmaas

Dam this is really cool thanks Mo!

Think we should also rename the folder to aries-cloudcontroller.

I also intend to go through all the notebook setups still and start to use the deployed pip package rather than local files for this.

wip-abramson avatar Apr 15 '21 10:04 wip-abramson