Skip to content

Commit

Permalink
Update README.md
Browse files Browse the repository at this point in the history
I'd like to report an inconsistency in the project requirements and test implementation:

The README instructions state:
"Add a function addPayedArtist() that takes as parameters a string and an address to register an artist to the festival."

However, the tests are only using the address parameter and not the name (string) parameter when calling this function.

This discrepancy is causing confusion and leading to implementation errors. It would be helpful to either:
1. Update the tests to include both the name and address parameters, or
2. Modify the README to reflect that only the address is required for the addPayedArtist() function.

Clarifying this point will ensure that the implementation matches the intended requirements and resolves the current test failures.
  • Loading branch information
abdotop authored Oct 10, 2024
1 parent b7df344 commit c0823e8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion subjects/blockchain/time-is-money/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ Also, in order for payments to go orderly, we decide that artists can get payed
- Create a Smart Contract `TimeIsMoney` that takes a uint as a parameter for its `constructor()` function that represents the date of the festival.
- Add a function `buyTicket()` that allows to buy one ticket for a minimum price of 0.01 ethers 10 days before its start, and 0.1 afterward.
- A function `ticketsOf()` that returns the number of tickets owned by that particular address.
- Add a function `addPayedArtist()` that takes as parameters a string and an address to register an artist to the festival.
- Add a function `addPayedArtist()` that takes as parameter an address to register an artist to the festival.
- Add a function `getPayed()` that allows an artist registered with the function above to receive 1 Ether, three days after the start of the festival.
- Add a function `getBenefits()` that can be triggered only by the organizer 10 days after the start of the festival. It receives the remainder of money from the festival.

Expand Down

0 comments on commit c0823e8

Please sign in to comment.