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

Send notification message to new project manager upon assignment and refactor assignment code #2016

Merged
merged 3 commits into from
Dec 26, 2024

Conversation

Anuj-Gupta4
Copy link
Collaborator

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

Refactored add manager endpoint to only allow organisation admins to assign managers
added code to send osm message and email to the new manager upon assignment

Screenshots

image

image

Review Guide

Verify project url would work on production.

Checklist before requesting a review

[optional] What gif best describes this PR or how it makes you feel?

… assignment and refactor add manager endpoint
@Anuj-Gupta4 Anuj-Gupta4 requested a review from Sujanadh December 23, 2024 08:04
@github-actions github-actions bot added docs Improvements or additions to documentation enhancement New feature or request backend Related to backend code labels Dec 23, 2024
@Anuj-Gupta4 Anuj-Gupta4 changed the title feat/send notification message to new project manager upon assignment and refactor assignment code Send notification message to new project manager upon assignment and refactor assignment code Dec 25, 2024
@Sujanadh Sujanadh merged commit bec592e into hotosm:development Dec 26, 2024
4 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend code docs Improvements or additions to documentation enhancement New feature or request tests Related to automated code tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Assign project admin during project creation
2 participants