Skip to content

feature: added the map key (legend) to the my forest section#1645

Merged
mohitb35 merged 5 commits into
developfrom
Feature/type_of_contributions
Feb 23, 2023
Merged

feature: added the map key (legend) to the my forest section#1645
mohitb35 merged 5 commits into
developfrom
Feature/type_of_contributions

Conversation

@sunilsabatp
Copy link
Copy Markdown
Contributor

@sunilsabatp sunilsabatp commented Dec 20, 2022

feature -> added map key to my forest section
visit to /profile and should have either register trees or donated trees for that login account

image

@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 20, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
planet-webapp ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Jan 25, 2023 at 7:18AM (UTC)

@mariahosfeld
Copy link
Copy Markdown
Contributor

Please change "Donated trees" to "Donated Trees". Other than that it looks good to me.

@mohitb35 mohitb35 changed the title feature: added the map key to the my forest section feature: added the map key (legend) to the my forest section Dec 21, 2022
Copy link
Copy Markdown
Collaborator

@norbertschuler norbertschuler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGFM, I pushed some commit to add a German translation, but as the branch name used uppercase characters Feature/type_of_contributions and my push used the standard lowercase name, GitHub could not correctly associate it with this PR.

Comment thread src/features/user/Profile/components/MyTrees/MyTrees.tsx
Copy link
Copy Markdown
Member

@mohitb35 mohitb35 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to merge.

@mohitb35 mohitb35 merged commit 15c73a0 into develop Feb 23, 2023
@mohitb35 mohitb35 deleted the Feature/type_of_contributions branch February 23, 2023 11:10
@pr-triage pr-triage Bot added the PR: merged label Feb 23, 2023
@mohitb35 mohitb35 mentioned this pull request Feb 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants