Skip to content

fix(dashboard): fix wordings for project types and its labels#6

Merged
shreeyash07 merged 3 commits into
developfrom
fix/project-types
Sep 4, 2025
Merged

fix(dashboard): fix wordings for project types and its labels#6
shreeyash07 merged 3 commits into
developfrom
fix/project-types

Conversation

@subinasr
Copy link
Copy Markdown
Contributor

@subinasr subinasr commented Sep 4, 2025

Changes

  • Fix labels and subheadings of various project types in stats card in the dashboard
  • Add 'STREET' project type in dashboard statistics

This PR doesn't introduce any:

  • temporary files, auto-generated files or secret keys
  • build works
  • eslint issues
  • typescript issues
  • codegen errors
  • console.log meant for debugging
  • typos
  • unwanted comments
  • conflict markers

Comment thread app/views/StatsBoard/index.tsx Outdated
</div>
)}
subHeading="Find"
subHeading="Find features"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's make this consistent with other subheadings, Find features should be Find Features (capital F)

Comment thread app/views/StatsBoard/index.tsx Outdated
label={(
<div className={styles.infoLabel}>
Streets Covered
Streets Viewed
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should be Street Viewed

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Are you sure??

@subinasr subinasr requested a review from kopitek8 September 4, 2025 10:49
@shreeyash07 shreeyash07 merged commit c7e5ec8 into develop Sep 4, 2025
1 check passed
@shreeyash07 shreeyash07 deleted the fix/project-types branch September 4, 2025 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants