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

Merge feature branch graph-agent into master #25

Merged
merged 219 commits into from
Jan 25, 2021
Merged

Merge feature branch graph-agent into master #25

merged 219 commits into from
Jan 25, 2021

Conversation

ynx0
Copy link
Owner

@ynx0 ynx0 commented Jan 25, 2021

The biggest update yet.

Clocking in at over 200 commits, this PR brings into fruition the completion of milestone 2, the client-side implementation of %graph-store.

Changelog:

  • implemented port of Landscape's graph store api

  • added custom exceptions and exception hierarchy for Airlock

  • performed refactoring, cleanup, and deduplication of code in AirlockChannel

  • added ability to perform a manual ota on fakezod while setting up test environment

  • added basic unit tests for graph-store

  • added AirlockCreditials class

  • updated Main.java example to use graph-store

  • documentation for the new classes

- Basically, we divide up the tests into different categories
  1. Core
  2. GraphStore
  3. ChatStore (will be depricated anyways)
  This helps us keep each file small enough to be readable and organized.
- also rename some files
- Update physical structure for organization
- add notes behind design thoughts of exception
- clean up exception hierarchy
- update code to reflect new exceptions
- this design isn't finalized, but its just mimicking the Landscape api
- implement `createUnmanagedGraph`
- renamings
- added todos
@ynx0 ynx0 merged commit f67697d into master Jan 25, 2021
@ynx0 ynx0 deleted the graph-agent branch January 25, 2021 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant