This is necessary but insufficient. If you use role=button, it needs to behave like a button, which means that Space and Enter must also activate it.
Things to read:
https://www.w3.org/TR/wai-aria-practices/#read_me_first
https://www.w3.org/TR/wai-aria-practices/#button