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.

Sign up for GitHub

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

Jump to bottom

Assign PM during project creation & Refactor #2115

Merged
merged 19 commits into from
Jan 27, 2025
Merged

Assign PM during project creation & Refactor #2115

merged 19 commits into from
Jan 27, 2025

Conversation

NSUWAL123
Copy link
Contributor

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

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation
  • 🧑‍💻 Refactor
  • ✅ Test
  • 🤖 Build or CI
  • ❓ Other (please specify)

Related Issue

Describe this PR

This PR contains the following works:

  • Addition of new select component comprising single, multi-select behaviour with search functionality
  • Separating radix components on a separate folder for separation of concerns
  • User slice, service, types & store integration
  • Common search bar component add
  • Create project: multi-select add for assigning project managers during project creation

Screenshots

Screenshot from 2025-01-24 15-43-03
Screenshot from 2025-01-24 15-42-45
Screenshot from 2025-01-24 15-29-36

@NSUWAL123
feat(package): radix popover and cmdk add
e8f4ab3
@NSUWAL123
feat(icons): icons add
a140b17
@NSUWAL123
feat(useDebouncedInput): useDebouncedInput hook add
a995bac
@NSUWAL123
feat(command): command radix component add
47fc6fe
@NSUWAL123
feat(dialog): dialog radix component add
4354930
@NSUWAL123
feat(popover): popover radix component add
b1d06f0
@NSUWAL123
feat(select): select radix component add
3d56daf
@NSUWAL123
feat(user): user slice, types, service setup & store integration
c6df350
@NSUWAL123
fix(createProjectService): finally block add to change organization l…
5c1137f 
…ist loading state
@NSUWAL123
fix(createProjectModel): remove organizationDropdownType
9d2570e
@NSUWAL123
feat(select2): new select component with single, multi-select & searc…
5d224c1 
…h functionality
@NSUWAL123
feat(searchBar): common search bar component
b301b81
@NSUWAL123
feat(select2): show selected values in capsules on multi-select case
798e1dc
@NSUWAL123
fix(type): ts type update
51b02a9
@NSUWAL123
feat(projectDetailsForm): replace previous select component with new …
6da5937 
…select component
@NSUWAL123
refactor(splitTasks): pass project admins as an argument on project c…
3420beb 
…reation action call
@NSUWAL123
feat(createProjectService): assign project manager during project cre…
581b3e7 
…ation
@NSUWAL123
fix(projectDetailsForm): update field text
ad8c82c
@NSUWAL123
Merge branch 'development' of github.com:hotosm/fmtm into feat/assign…
7301d52 
…-project-admin
@NSUWAL123 NSUWAL123 requested a review from spwoodcock January 24, 2025 10:17
@github-actions github-actions bot added enhancement New feature or request frontend Related to frontend code labels Jan 24, 2025
spwoodcock
spwoodcock approved these changes Jan 27, 2025
View reviewed changes
Copy link
Member

@spwoodcock spwoodcock left a comment

Choose a reason for hiding this comment

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

Wow this change is huge - seriously good work 🎉

@spwoodcock spwoodcock merged commit 69a9973 into development Jan 27, 2025
6 checks passed
@spwoodcock spwoodcock deleted the feat/assign-project-admin branch January 27, 2025 20:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@spwoodcock spwoodcock spwoodcock approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request frontend Related to frontend code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@NSUWAL123 @spwoodcock