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

FIP-0076 Add Market::GetDealSector method #852

Merged
merged 2 commits into from
Nov 8, 2023
Merged

FIP-0076 Add Market::GetDealSector method #852

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
773ad9f
FIP-0076 Add Market::GetDealSector method
anorth Oct 18, 2023
d9d4ffc
review
anorth Oct 31, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions FIPS/fip-0076.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -542,6 +542,25 @@ struct OnMinerSectorsTerminateParams {
The sector is removed from the `ProviderSectors` mapping, if present.
Any deals mapped to that sector are marked as terminated, and subsequent processing deferred to cron.

#### GetDealSector

A new FRC-0042 exported method "GetDealSector" (method 2611213344) returns the sector number in which
a deal's data is stored, while a deal is active.
If a deal is published but not yet activated, aborts with `EX_DEAL_NOT_ACTIVATED = 33`.
If a deal is not found, aborts with the same exit codes as "GetDealActivation".

```
// "Transparent" CBOR-encoding, the singleton field is encoeded directly.
struct GetDealSectorParams {
DealID: DealID,
}

// "Transparent" CBOR-encoding, the singleton field is encoeded directly.
struct GetDealSectorReturn {
SectorNumber: SectorNumber,
}
```

### Migration

The built-in market actor's `ProviderSectors` mapping is initialised from the existing deal state
Expand Down