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

feat: CID gravity connector #407

Merged
merged 15 commits into from
Feb 27, 2025
Merged

feat: CID gravity connector #407

merged 15 commits into from
Feb 27, 2025

Conversation

LexLuthr
Copy link
Contributor

@LexLuthr LexLuthr commented Feb 10, 2025

  • Enable connector
  • Send failure count
  • Add CIDGravity to UI
  • Test in mainnet

A few rejected deals:

cid gravity response {"dealId": "b46ad7fd-caea-4aa5-a26b-9fbab37196ce", "decision": "reject", "customMessage": "", "externalMessage": "CLIENT_NOT_AUTHORIZED: this address is not allowed to send deals", "internalMessage": "CLIENT_NOT_AUTHORIZED: deals from unknown clients are disabled", "externalDecision": "clientNotAuthorized", "internalDecision": "reject", "matchingAcceptanceLogic": "", "matchingPricing": "", "matchingRule": -1}

cid gravity response {"dealId": "b6064064-7263-4db8-a559-ee42e40fc826", "decision": "reject", "customMessage": "", "externalMessage": "PRICE_TOO_LOW: accepted price for such deal: 0.00000002 FIL/Epoch/GiB", "internalMessage": "PRICE_TOO_LOW: matched pricing model Sample pricing / Rule 3", "externalDecision": "priceTooLow", "internalDecision": "reject", "matchingAcceptanceLogic": "", "matchingPricing": "Sample pricing", "matchingRule": 3}

cid gravity response {"dealId": "196a4f0f-0006-4517-8f0e-5ea912e194e1", "decision": "reject", "customMessage": "", "externalMessage": "BUSY: please retry later", "internalMessage": "BUSY: deal rejected by storage acceptance logic: 'Test curio'", "externalDecision": "busy", "internalDecision": "storageAcceptanceLogic", "matchingAcceptanceLogic": "", "matchingPricing": "", "matchingRule": -1}

Deal accepted :

cid gravity response {"dealId": "331f59e2-babf-4487-b099-2c2d46a86e06", "decision": "accept", "customMessage": "", "externalMessage": "ACCEPTED: deal accepted", "internalMessage": "ACCEPTED: deal accepted by pricing model Sample pricing / Rule 3", "externalDecision": "accept", "internalDecision": "accept", "matchingAcceptanceLogic": "Test curio", "matchingPricing": "Sample pricing", "matchingRule": 3}

Market Settings page

Screenshot 2025-02-27 at 3 42 01 PM Screenshot 2025-02-27 at 4 09 15 PM

@LexLuthr LexLuthr force-pushed the feat/cid-gravity branch 2 times, most recently from 09d3f58 to 854ad9f Compare February 10, 2025 16:44
@LexLuthr LexLuthr requested a review from magik6k February 11, 2025 16:15
Copy link
Collaborator

@magik6k magik6k left a comment

Choose a reason for hiding this comment

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

Looks good assuming that it works.

Could we add some navigation from our webui (e.g. in Market Settings) to the cidgravity dashboard when the settings enable the integration?

@LexLuthr
Copy link
Contributor Author

I will add the navigation page from market page and even display that CIDgravity is enabled for deal filtering. There is one last bit pending from our side about failure count and then testing.

@LexLuthr LexLuthr marked this pull request as ready for review February 27, 2025 12:18
@LexLuthr LexLuthr merged commit 395bc47 into main Feb 27, 2025
31 checks passed
@LexLuthr LexLuthr deleted the feat/cid-gravity branch February 27, 2025 13:42
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