austintgriffith / bouncer-proxy Goto Github PK

Hello @austintgriffith ,
I am trying to migrate the contract to the latest Solidity version. My understanding is that 0.5.0+ enforces stronger types. I managed to fix most errors except one, line 90.
I replaced this require(msg.sender.call.value(rewardAmount).gas(36000)()); with require(msg.sender.call.value(rewardAmount).gas(36000)("")); as recommended in the breaking changes section of the docs.
However I am still returned the following error : TypeError: No matching declaration found after argument-dependent lookup. require(msg.sender.call.value(rewardAmount).gas(36000)(""));
I am not sure what the problem is. I tried to search on S.O & Goog and from the different posts I read, it seems related to parameters not being of the good type.
I am a bit clueless as to how to fix that error though.
Do you have any ideas ?
Many thanks.

Bulk vulnerability fix - Lockfile fix
This pull request will update your transitive dependencies within the allowed version intervals provided by your direct dependencies.

Fixed vulnerabilities:
CVE–2018–16469 debricked

CVE–2018–6341 debricked

CVE–2018–6342 debricked

CVE–2019–10746 debricked

CVE–2019–10747 debricked

CVE–2019–19919 debricked

CVE–2019–20149 debricked

CVE–2020–8124 debricked

CVE–2020–7662 debricked

CVE–2017–18869 debricked

CVE–2020–7733 debricked

CVE–2019–20920 debricked

CVE–2019–20922 debricked

CVE–2020–7788 debricked

CVE–2020–7793 debricked

CVE–2020–26291 debricked

debricked–124

debricked–149414

debricked–149661

debricked–149740

debricked–149815

debricked–149816

debricked–149824

CVE–2021–27515 debricked

CVE–2021–27516 debricked

CVE–2021–27292 debricked

CVE–2021–23362 debricked

CVE–2021–23369 debricked

CVE–2021–23383 debricked

CVE–2021–23343 debricked

CVE–2021–23386 debricked

CVE–2021–3647 debricked

CVE–2021–3664 debricked

CVE–2021–32804 debricked

CVE–2021–32803 debricked

CVE–2021–37713 debricked

CVE–2021–37712 debricked

CVE–2021–37701 debricked

CVE–2021–23440 debricked

CVE–2021–3777 debricked

CVE–2021–3807 debricked

CVE–2021–3803 debricked

CVE–2021–3918 debricked

CVE–2022–0512 debricked

CVE–2022–0613 debricked

CVE–2022–0639 debricked

CVE–2022–0691 debricked

CVE–2022–0686 debricked

CVE–2022–24723 debricked

CVE–2022–0868 debricked

CVE–2022–1233 debricked

CVE–2022–1243 debricked

CVE–2022–21222 debricked

CVE–2022–37599 debricked

CVE–2022–24999 debricked

CVE–2022–38900 debricked

CVE–2022–25927 debricked

CVE–2022–38778 debricked

Watch my merged pull request from my site and give me a star =)

I always loose track of the blog post when navigating here.
I'd be great if this could be added to the readme:
https://medium.com/@austin_48503/ethereum-meta-transactions-90ccf0859e84