Skip to content

dex-api migration for two-way orders#92

Open
irfanaligy wants to merge 14 commits intogeniusyield:mainfrom
irfanaligy:irfanaligy/migrate-Core
Open

dex-api migration for two-way orders#92
irfanaligy wants to merge 14 commits intogeniusyield:mainfrom
irfanaligy:irfanaligy/migrate-Core

Conversation

@irfanaligy
Copy link
Contributor

PR draft to review dex-api migration for SLV

  • geniusyield-dex-api, geniusyield-core-common migrated to match Core structure
  • relevant helpers for smart-order routers added in geniusyield-dex-api
  • geniusyield-server-api migrated

@irfanaligy irfanaligy requested a review from a team as a code owner November 28, 2025 18:32
@irfanaligy irfanaligy marked this pull request as draft November 28, 2025 18:33
@irfanaligy irfanaligy marked this pull request as ready for review December 19, 2025 15:00
@vjk0
Copy link

vjk0 commented Dec 24, 2025

The runtime Docker build doesn't contain all the required dependencies and tries to look for them in a shared cabal directory that is not relevant to the target runtime:
geniusyield-server: /root/.cabal/share/x86_64-linux-ghc-9.6.5/geniusyield-dex-api-0.1.0.0/DEX.NFT: withBinaryFile: does not exist (No such file or directory)

Copy link
Contributor

@4TT1L4 4TT1L4 left a comment

Choose a reason for hiding this comment

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

LGTM.

@brunjlar please take a look as well.

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.

3 participants