search

kokarn / tarkov-tools

https://tarkov-tools.com
MIT License
70 stars 44 forks source link

Flea fee reduction on 3rd lvl of int. center #101

Closed ghost closed 3 years ago

ghost commented 3 years ago

Noticed this issue so i made this small change

netlify[bot] commented 3 years ago

✔️ Deploy Preview for tarkov-tools-preview ready!

🔨 Explore the source changes: 250dad860c09b6868ceaf318d5567e9f7c727c55

🔍 Inspect the deploy log: https://app.netlify.com/sites/tarkov-tools-preview/deploys/617a839c3b70ca0008cc930e

😎 Browse the preview: https://deploy-preview-101--tarkov-tools-preview.netlify.app/

kokarn commented 3 years ago

Overall this looks great!

There seems to be 2 small errors

8:31:33 PM: src/components/crafts-table/index.js
8:31:33 PM:   Line 50:58:  Expected '===' and instead saw '=='                                                                            eqeqeq
8:31:33 PM:   Line 266:9:  React Hook useMemo has a missing dependency: 'feeReduction'. Either include it or remove the dependency array  react-hooks/exhaustive-deps

Would you mind fixing those?

kokarn commented 3 years ago

Thanks so much!