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
Resolve TS errors: TreeMapper #1860
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@prachigarg19 Please resolve conflicts. Is this otherwise complete, or is anything left? |
1. add type for api call 2. resolve `index.tsx` 3. update import in `my-species.tsx`
It's complete |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@prachigarg19 Please go through my feedback
src/features/user/TreeMapper/Analytics/components/TreePlanted/index.tsx
Outdated
Show resolved
Hide resolved
src/features/user/TreeMapper/Import/components/ReviewSubmit.tsx
Outdated
Show resolved
Hide resolved
src/features/user/TreeMapper/Import/components/PlantingLocation.tsx
Outdated
Show resolved
Hide resolved
src/features/user/TreeMapper/Import/components/PlantingLocation.tsx
Outdated
Show resolved
Hide resolved
src/features/user/TreeMapper/Import/components/PlantingLocation.tsx
Outdated
Show resolved
Hide resolved
1. Rename SpeciesType and move it to plantLocations.d.ts 2. Rename newSpecies interface
1. Resolve csv error 2. Add comment for csv validation
1. add condition for null plantLocation 2. Update props to remove null in plantLocations 3. Remove treemapper error
src/features/user/TreeMapper/Import/index.tsx - specify type for api handler using <T> format src/features/user/TreeMapper/Import/components/SampleTreeCard.tsx - remove usage of Treemapper namespace - resolve warnings in rhf components
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM now.
This PR resolves typescript errors