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.

Sign up for GitHub

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

Jump to bottom

Implementation core with error handling and protocol adapter #19

Closed
fuxingloh opened this issue Feb 26, 2021 · 0 comments · Fixed by #41
Closed

Implementation core with error handling and protocol adapter #19

fuxingloh opened this issue Feb 26, 2021 · 0 comments · Fixed by #41
Labels
kind/feature New feature request priority/important-soon Will be important soon triage/accepted Triage has been accepted
Milestone
Jellyfish Protocols

Comments

@fuxingloh
Copy link
Contributor

What would you like to be added:

jellyfish-core implementation must be:

  1. modern with near zero dependencies to mitigate risk
  2. support interceptors implementation but not implemented on day one
  3. error handling at core, Promise.reject()
  4. ability to adapt to any protocol when introduced, ws, https etc while being simple to use

/triage accepted
/area core
@fuxingloh fuxingloh added the kind/feature New feature request label Feb 26, 2021
@defichain-bot defichain-bot added triage/accepted Triage has been accepted area/core labels Feb 26, 2021
@fuxingloh fuxingloh changed the title core with interceptors, error handling and protocol adapter Implementation core with error handling and protocol adapter Mar 1, 2021
@fuxingloh fuxingloh added this to the Jellyfish Protocols milestone Mar 2, 2021
@fuxingloh fuxingloh mentioned this issue Mar 2, 2021
Closed
@fuxingloh fuxingloh added the priority/important-soon Will be important soon label Mar 5, 2021
@fuxingloh fuxingloh mentioned this issue Mar 8, 2021
Merged
@fuxingloh fuxingloh linked a pull request Mar 8, 2021 that will close this issue
jellyfish-core protocol adapter and error handling #41
Merged
eli-lim pushed a commit that referenced this issue Mar 27, 2022
@fuxingloh
added provider agnostic database module (#19)
155f584 
* updated jellyfish-api-* versions
* added database module abstraction
* e2e test will use database memory provider by default
* added missing WHALE_DATABASE_PROVIDER for sanity testing
canonbrother pushed a commit that referenced this issue Jun 1, 2022
@fuxingloh
added provider agnostic database module (#19)
3aa0613 
* updated jellyfish-api-* versions
* added database module abstraction
* e2e test will use database memory provider by default
* added missing WHALE_DATABASE_PROVIDER for sanity testing
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/feature New feature request priority/important-soon Will be important soon triage/accepted Triage has been accepted
Projects
None yet
Milestone
Jellyfish Protocols
Development

Successfully merging a pull request may close this issue.

jellyfish-core protocol adapter and error handling BirthdayResearch/jellyfishsdk
2 participants
@fuxingloh @defichain-bot