Skip to content
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

Feat ts type add organization create project #1293

Conversation

NSUWAL123
Copy link
Contributor

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Related Issue

Describe this PR

This PR adds the following things in files related to the organization & project-details:

  1. Add missing TS types
  2. Replace import of useAppSelector from CoreModules to reduxTypes as it provides types that has been set to the reduxSlice by default & no necessity to again add types on the component.

Screenshots

20240228_113253

@github-actions github-actions bot added the frontend Related to frontend code label Feb 28, 2024
@NSUWAL123
Copy link
Contributor Author

Also added types to the remaining components and used useAppSelector from reduxType instead of coreModules

@varun2948 varun2948 merged commit 2857f6c into hotosm:development Feb 28, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Related to frontend code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants