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

[Bug] Can't accept self-created offer files #14892

Closed
HarveyPeachey opened this issue Mar 23, 2023 · 5 comments
Closed

[Bug] Can't accept self-created offer files #14892

HarveyPeachey opened this issue Mar 23, 2023 · 5 comments
Labels
1.7.1 bug Something isn't working GUI Related to the Chia GUI NFT offers RPC Pull requests that change or introduce RPC commands stale-issue flagged as stale and will be closed in 7 days if not updated websocket Windows Impacts Windows

Comments

@HarveyPeachey
Copy link

What happened?

I was attempting to test Dexie's websocket protocol using testnet and tried to self-accept an NFT offer to test the emitting mechanism, however, I get an error in the GUI stating Trade for this offer already exists.. I have also encountered the same error using the Wallet RPC take_offer.

I've asked about this in the support channel on Keybase, and a kind Kiwi asked the engineering team, and said:

the offer accept code was modified to add checks to ensure you don't accept the same offer twice. I appears the code is looking at all transactions so it's seeing both the offer and the accept and seeing it as a duplicate transaction, hence the message

However, I would like the ability back to self-accept offers for testing purposes, even if it's just on testnet as I think this was an unintentional consequence of the aforementioned changes.

Version

1.7.1

What platform are you using?

Windows

What ui mode are you using?

GUI

Relevant log output

Error
Trade for this offer already exists.
@HarveyPeachey HarveyPeachey added the bug Something isn't working label Mar 23, 2023
@MumfMeisterT MumfMeisterT added RPC Pull requests that change or introduce RPC commands GUI Related to the Chia GUI Windows Impacts Windows offers NFT websocket 1.7.1 labels Mar 30, 2023
@MumfMeisterT
Copy link

Are you offering an NFT? And what are you asking for in return for the offer?

@paninaro
Copy link
Contributor

Snippet from my debug log:

  File "/Users/jeffc/src/macos/chia-blockchain/chia/rpc/rpc_server.py", line 340, in safe_handle
    response = await self.ws_api(message)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/jeffc/src/macos/chia-blockchain/chia/rpc/rpc_server.py", line 331, in ws_api
    return await f_rpc_api(data)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/Users/jeffc/src/macos/chia-blockchain/chia/rpc/wallet_rpc_api.py", line 1688, in take_offer
    trade_record, tx_records = await self.service.wallet_state_manager.trade_manager.respond_to_offer(
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/jeffc/src/macos/chia-blockchain/chia/wallet/trade_manager.py", line 788, in respond_to_offer
    await self.save_trade(trade_record, offer.name())
  File "/Users/jeffc/src/macos/chia-blockchain/chia/wallet/trade_manager.py", line 386, in save_trade
    await self.trade_store.add_trade_record(trade, offer_name)
  File "/Users/jeffc/src/macos/chia-blockchain/chia/wallet/trading/trade_store.py", line 164, in add_trade_record
    raise ValueError("Trade for this offer already exists.")
ValueError: Trade for this offer already exists.

@HarveyPeachey
Copy link
Author

Are you offering an NFT? And what are you asking for in return for the offer?

Offering an NFT for XCH

Copy link
Contributor

This issue has not been updated in 14 days and is now flagged as stale. If this issue is still affecting you and in need of further review, please comment on it with an update to keep it from auto closing in 7 days.

@github-actions github-actions bot added the stale-issue flagged as stale and will be closed in 7 days if not updated label Dec 21, 2024
Copy link
Contributor

This issue was automatically closed because it has been flagged as stale, and subsequently passed 7 days with no further activity from the submitter or watchers.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Dec 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1.7.1 bug Something isn't working GUI Related to the Chia GUI NFT offers RPC Pull requests that change or introduce RPC commands stale-issue flagged as stale and will be closed in 7 days if not updated websocket Windows Impacts Windows
Projects
None yet
Development

No branches or pull requests

5 participants