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

If a contract has no callable actions, but has a notification handler, the notification handler doesn't work #156

Open
mdcryptonfts opened this issue Apr 27, 2023 · 0 comments
Labels
bug Something isn't working good first issue Good for newcomers triage

Comments

@mdcryptonfts
Copy link

I recently created a contract that had some on_notify logic in it.

The contract had no callable actions though, just the on_notify.

As expected, compiling the contract gave me a couple warnings about a blank contract, and said that no ABI was generated.

I pushed this contract to the chain, and the notification handler did not work.

I needed to add a "dummy action" to the contract, in order to generate an ABI. Once I did this, the notification handler also worked properly.

It would likely be useful if one could compile this same type of contract without needing to add the dummy action in order to generate an ABI and get the notification handler to work.

@stephenpdeos stephenpdeos added bug Something isn't working 👍 lgtm and removed triage labels May 9, 2023
@BenjaminGormanPMP BenjaminGormanPMP added the good first issue Good for newcomers label May 25, 2023
@ericpassmore ericpassmore added this to the CDT 5.0.0-rc1 milestone Aug 11, 2023
@ericpassmore ericpassmore modified the milestones: CDT 4.1.0-rc1, Future Milestone Aug 24, 2023
@arhag arhag removed this from the Future Milestone milestone Dec 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers triage
Projects
Status: Icebox
Development

No branches or pull requests

6 participants