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

Implement receive function to handle received ETH in Ethlance.sol #443

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

afrasiyabhaider
Copy link

Summary

implements the TODO in the receive function of the Ethlance.sol contract to handle received ETH. The function constructs _offeredValues from msg.value and calls _createJobWithPassedData with the decoded msg.data.

Changes:

Implement receive function to handle received ETH. Construct _offeredValues from msg.value.
Call _createJobWithPassedData with the decoded msg.data.

Impact:
This change ensures that the contract correctly handles incoming ETH and creates jobs accordingly, improving the functionality and robustness of the Ethlance platform.

Review notes

Testing notes

 implements the TODO in the receive function of the Ethlance.sol contract to handle received ETH. The function constructs _offeredValues from msg.value and calls _createJobWithPassedData with the decoded msg.data.

Changes:

Implement receive function to handle received ETH.
Construct _offeredValues from msg.value.
Call _createJobWithPassedData with the decoded msg.data.

Impact:
This change ensures that the contract correctly handles incoming ETH and creates jobs accordingly, improving the functionality and robustness of the Ethlance platform.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant