8d ago in Code Review
PR #29
check please
Patch Notes
- User can now add two coins to the main page graphs. If they try and add more than two it won't let them. Clicking a coin already selected will remove that coin. Still need to clean this up but the feature is finally there!
- Fixed bug where tailwind wasn't taking into account files that I labeled ".jsx" in the configuration
- Removed variable in the "addToGraph" function of main page. Not needed as the information needed was already elsewhere
- Refactored two large blocks of code on main page. Moved the buttons for the graph into "MainGraphDaySelection.jsx" and moved the header for the bottom coin list into "BottomCoinHeader.jsx".
- Renamed CoinDetails to BottomCoinList and CoinStatistics to TopCoinList. Just wanted to make it easier to know what they are.
- Various tailwind/css updates
- Starting to add comments explaining some functions. Probably should have done this earlier but these will be added in periodically.
- Updated contact page so that clicking my email will open the option to email me
4
0 comments
PR #29
skool.com/developerpro
Learn how to code. Make money. Have fun. Enjoy life.
Leaderboard (30-day)
1
+234
2
+99
3
+87
4
+81
5
+78
powered by