issues
search
kwight
/
blockchain-developer-bootcamp-final-project
MIT License
2
stars
0
forks
source link
issues
Newest
Newest
Most commented
Recently updated
Oldest
Least commented
Least recently updated
Templates are inconsistently named.
#33
kwight
opened
2 years ago
0
The public getters do not have boundaries on the amount of data returned.
#32
kwight
opened
2 years ago
0
The `readme.md` is missing a screengrab of the project.
#31
kwight
closed
2 years ago
0
The owner view form is not reset on submission.
#30
kwight
closed
2 years ago
1
Remove Charity buttons should not be active for non-owners.
#29
kwight
closed
2 years ago
1
Donate directly to a charity's program.
#28
kwight
closed
2 years ago
0
Programs could be better named as "campaigns".
#27
kwight
opened
2 years ago
0
Charity rendering does not automatically update from on-chain activity.
#26
kwight
closed
2 years ago
1
The wallet's current network is not verified to be correct.
#25
kwight
opened
2 years ago
2
The contract does not use a withdrawal pattern.
#24
kwight
opened
2 years ago
0
Timestamp tests can't be run reliably without bumping times more than necessary.
#23
kwight
closed
2 years ago
1
The project has some magic numbers.
#22
kwight
closed
2 years ago
1
An event's charity cannot block a participant.
#21
kwight
closed
2 years ago
1
Events from removed charities are not deleted.
#20
kwight
closed
2 years ago
1
Event Cancel buttons are active for connections that didn't register that event.
#19
kwight
closed
2 years ago
1
A charity's events are not marked `cancelled` after that charity is removed.
#18
kwight
closed
2 years ago
1
Compiled artifacts aren't easily available to the client.
#17
kwight
opened
2 years ago
1
Investigate global error handling with ethers events.
#16
kwight
opened
2 years ago
0
Registered charities should be unique in `charitiesList`.
#15
kwight
closed
2 years ago
1
`charities` is not a fixed array length.
#14
kwight
closed
2 years ago
1
Scripts have no cache busting.
#13
kwight
opened
2 years ago
0
Current address UI is duplicated on account switching.
#12
kwight
closed
2 years ago
0
`renderAccountData`'s parameter needs better handling.
#11
kwight
closed
2 years ago
1
The error state of `getConnectedAccount` needs handling.
#10
kwight
closed
2 years ago
1
MetaMask provider detection is handled locally.
#9
kwight
opened
2 years ago
0
All functions are lacking docblocks.
#8
kwight
opened
2 years ago
1
Verify appropriate use of modifiers and in-function `require` statements.
#7
kwight
closed
2 years ago
1
Events are not cancelled when a charity is removed.
#6
kwight
opened
2 years ago
0
Access control is not role-based.
#5
kwight
opened
2 years ago
1
Contracts have not been scanned for known vulnerabilities.
#4
kwight
closed
2 years ago
1
Verify `memory` versus `calldata` usage.
#3
kwight
opened
2 years ago
0
Verify consistent usage of underscored variables.
#2
kwight
opened
2 years ago
0
Contract methods are not emitting events.
#1
kwight
closed
2 years ago
1