Stefan Stefanov
Stefan Stefanov
> I wonder if this use case is handled by [this](https://github.com/hashgraph/hedera-services/pull/12842/files) pr? That was a good catch. We synced with the Merge Request authors and we are figuring out a...
> LG overall. A few comments that you can address or not at your discretion. The new version is ready for a review @lukelee-sl. Thank you
> Seems reasonable but should there not be a HAPI test to show that the transaction reverts as outlined in the original issue? Hey @lukelee-sl Thank you for the review....
> In `ContractKeysHTSV1SecurityModelSuite` we have `burnTokenWithFullPrefixAndPartialPrefixKeys()` test that is not updated for v2. Is it on purpose? Hey, @petreze I think it is.... the new name is `V2SecurityBurnTokenWithFullPrefixAndPartialPrefixKeys`. Am I...
> The changes should be in `GrantRevokeKycV1SecurityModelSuite` to enable and fix `grantRevokeKycSpec()` test I think the new test should be in `GrantRevokeKycSuite` file. The file name with the `v1` tag...