commerce-onchain-payment-protocol icon indicating copy to clipboard operation
commerce-onchain-payment-protocol copied to clipboard

update TransferIntent id type to bytes32

Open kylekaplan opened this issue 1 year ago • 31 comments

Hi,

I was wondering if the id field of the TransferIntent could be updated to a bytes32 instead of bytes16.

This would make it so the Keccak256 hash function could be used to generate predictable ids and be more compatible with other protocols generating ids with Keccak256.

For my use case, I'm using EAS (Ethereum Attestation Service) as a backend to create an on-chain eCommerce solution, and it would be nice if I could use an attestation UID (a Keccak256 hash of the attestation) as the id for the TransferIntent. This would make the relationship between an attestation and a TransferIntent simpler and more efficient.


Thanks for your consideration and for this awesome protocol.

kylekaplan avatar Aug 05 '24 23:08 kylekaplan

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

cb-heimdall avatar Aug 05 '24 23:08 cb-heimdall

Agreed

nichechristie avatar Aug 06 '24 00:08 nichechristie

Review Error for TELLYWHALE @ 2024-08-11 00:05:58 UTC User must have write permissions to review

cb-heimdall avatar Aug 11 '24 00:08 cb-heimdall

Review Error for TELLYWHALE @ 2024-08-11 00:30:59 UTC User must have write permissions to review

cb-heimdall avatar Aug 11 '24 00:08 cb-heimdall

Review Error for Nichebiche @ 2024-08-27 21:09:33 UTC User must have write permissions to review

cb-heimdall avatar Aug 27 '24 21:08 cb-heimdall

Review Error for MTerrases22 @ 2024-09-21 22:09:21 UTC User must have write permissions to review

cb-heimdall avatar Sep 21 '24 22:09 cb-heimdall

Does this mean I now have write access and permissions ? 

Sent from Yahoo Mail for iPhone

On Saturday, September 21, 2024, 6:09 PM, Heimdall @.***> wrote:

Review Error for MTerrases22 @ 2024-09-21 22:09:21 UTC User must have write permissions to review

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

nichechristie avatar Sep 21 '24 22:09 nichechristie

Review Error for Nichebiche @ 2024-09-21 22:14:21 UTC User must have write permissions to review

cb-heimdall avatar Sep 21 '24 22:09 cb-heimdall

Review Error for Irasone @ 2024-09-21 22:43:13 UTC User must have write permissions to review

cb-heimdall avatar Sep 21 '24 22:09 cb-heimdall

Review Error for Irasone @ 2024-09-21 22:44:29 UTC User must have write permissions to review

cb-heimdall avatar Sep 21 '24 22:09 cb-heimdall

Why haven't I received payment? I've been mistreated here. This is my account and no one should be preventing me from it. I'm so not letting this one go . I can't retrieve my funds. Someone took my identity and keeps locking me out of my account. I want my account left alone. It's my money not anyone else's.

MTerrases22 avatar Sep 23 '24 19:09 MTerrases22

What am I supposed to do with this

Get Outlook for Androidhttps://aka.ms/AAb9ysg


From: Sturgeontara06 @.> Sent: Thursday, October 10, 2024 5:17:24 PM To: coinbase/commerce-onchain-payment-protocol @.> Cc: Marlyn Elizabeth Terrases @.>; Comment @.> Subject: Re: [coinbase/commerce-onchain-payment-protocol] update TransferIntent id type to bytes32 (PR #18)

@Sturgeontara06 commented on this pull request.

Sure

— Reply to this email directly, view it on GitHubhttps://github.com/coinbase/commerce-onchain-payment-protocol/pull/18#pullrequestreview-2361656555, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BKBCEVRTDEKIT4I26JEKKLTZ24KJJAVCNFSM6AAAAABMBGYTBKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRRGY2TMNJVGU. You are receiving this because you commented.Message ID: @.***>

MTerrases22 avatar Oct 11 '24 00:10 MTerrases22

Review Error for Irasone @ 2024-10-11 01:46:04 UTC User must have write permissions to review

cb-heimdall avatar Oct 11 '24 01:10 cb-heimdall

Please check now.

Was this taken care of?

On Thu, Oct 10, 2024 at 6:46 PM Heimdall @.***> wrote:

Review Error for Irasone @ 2024-10-11 01:46:04 UTC User must have write permissions to review

— Reply to this email directly, view it on GitHub https://github.com/coinbase/commerce-onchain-payment-protocol/pull/18#issuecomment-2406373162, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKBCEVV2W734MHHH7XEHXZDZ24UXFAVCNFSM6AAAAABMBGYTBKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBWGM3TGMJWGI . You are receiving this because you commented.Message ID: @.*** com>

-- Marlyn E.Terrases

MTerrases22 avatar Oct 12 '24 13:10 MTerrases22

Awesome, thank you so much!

On Thu, Oct 10, 2024 at 6:46 PM Irasone @.***> wrote:

@.**** approved this pull request.

— Reply to this email directly, view it on GitHub https://github.com/coinbase/commerce-onchain-payment-protocol/pull/18#pullrequestreview-2361759224, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKBCEVU2SJTN3OIAJ43B5MLZ24UW5AVCNFSM6AAAAABMBGYTBKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRRG42TSMRSGQ . You are receiving this because you commented.Message ID: <coinbase/commerce-onchain-payment-protocol/pull/18/review/2361759224@ github.com>

-- Marlyn E.Terrases

MTerrases22 avatar Oct 12 '24 13:10 MTerrases22

How can I access that check. With authorization of course. Been locked out and unable to claim my assets.someone please help me.

MTerrases22 avatar Oct 14 '24 21:10 MTerrases22

Hello, I haven't been getting my emails.

On Sat, Sep 21, 2024 at 3:44 PM Heimdall @.***> wrote:

Review Error for Irasone @ 2024-09-21 22:44:29 UTC User must have write permissions to review

— Reply to this email directly, view it on GitHub https://github.com/coinbase/commerce-onchain-payment-protocol/pull/18#issuecomment-2365345091, or unsubscribe https://github.com/notifications/unsubscribe-auth/BKBCEVT74XR3KDXOWRJQ76DZXXZGHAVCNFSM6AAAAABMBGYTBKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRVGM2DKMBZGE . You are receiving this because you commented.Message ID: @.*** com>

-- Marlyn E.Terrases

MTerrases22 avatar Oct 20 '24 19:10 MTerrases22

Review Error for Tinaochoa8462 @ 2024-10-24 07:38:24 UTC User must have write permissions to review

cb-heimdall avatar Oct 24 '24 07:10 cb-heimdall

Review Error for joeltato @ 2024-11-07 11:33:52 UTC User must have write permissions to review

cb-heimdall avatar Nov 07 '24 11:11 cb-heimdall

how get my funds

On Thu, Nov 7, 2024 at 5:34 AM Heimdall @.***> wrote:

Review Error for joeltato @ 2024-11-07 11:33:52 UTC User must have write permissions to review

— Reply to this email directly, view it on GitHub https://github.com/coinbase/commerce-onchain-payment-protocol/pull/18#issuecomment-2462005594, or unsubscribe https://github.com/notifications/unsubscribe-auth/BMUM4PVE4A55VG7KR4S35DTZ7NF3NAVCNFSM6AAAAABMBGYTBKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRSGAYDKNJZGQ . You are receiving this because you commented.Message ID: @.*** com>

joeltato avatar Nov 07 '24 11:11 joeltato

Review Error for zwiux42 @ 2024-12-06 22:54:23 UTC User must have write permissions to review

cb-heimdall avatar Dec 06 '24 22:12 cb-heimdall

Review Error for zwiux42 @ 2024-12-07 01:08:15 UTC User must have write permissions to review

cb-heimdall avatar Dec 07 '24 01:12 cb-heimdall

Review Error for Irasone @ 2024-12-08 10:51:13 UTC User must have write permissions to review

cb-heimdall avatar Dec 08 '24 10:12 cb-heimdall

Review Error for Irasone @ 2024-12-08 10:52:46 UTC User must have write permissions to review

cb-heimdall avatar Dec 08 '24 10:12 cb-heimdall

Review Error for justpayme3 @ 2025-01-23 21:37:29 UTC User must have write permissions to review

cb-heimdall avatar Jan 23 '25 21:01 cb-heimdall

Review Error for Irasone @ 2025-01-23 21:42:42 UTC User must have write permissions to review

cb-heimdall avatar Jan 23 '25 21:01 cb-heimdall

Review Error for Irasone @ 2025-01-23 21:48:23 UTC User must have write permissions to review

cb-heimdall avatar Jan 23 '25 21:01 cb-heimdall

Review Error for Nichebiche @ 2025-01-23 22:00:14 UTC User must have write permissions to review

cb-heimdall avatar Jan 23 '25 22:01 cb-heimdall

Just merge it so we can update the bytes to bytes32

nichechristie avatar Jan 23 '25 22:01 nichechristie

What about this

nichechristie avatar Jan 23 '25 22:01 nichechristie