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

fix: Add support to fetch onboarding extension request #2331

Merged
merged 3 commits into from
Jan 15, 2025

Conversation

pankajjs
Copy link
Member

@pankajjs pankajjs commented Jan 2, 2025

Date: 2 Jan, 2025

Developer Name: @pankajjs


Issue Ticket Number

Description

This PR fixes GET /request endpoint by introducing ONBOARDING type in validator and fetching onboarding extension requests by requestedBy field under enabled feature flag.

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

Screenshot 1 Screenshot 2025-01-08 at 12 41 46 AM Screenshot 2025-01-08 at 12 41 54 AM

Test Coverage

Screenshot 1 Screenshot 2025-01-08 at 1 41 35 AM Screenshot 2025-01-08 at 1 41 28 AM

Additional Notes

@pankajjs pankajjs force-pushed the fix/get-request branch 2 times, most recently from aade0fc to a292d17 Compare January 7, 2025 14:46
@pankajjs pankajjs changed the title Add support to fetch onboarding extension request (fix): Add support to fetch onboarding extension request Jan 8, 2025
@pankajjs pankajjs changed the title (fix): Add support to fetch onboarding extension request fix: Add support to fetch onboarding extension request Jan 8, 2025
@pankajjs pankajjs force-pushed the fix/get-request branch 3 times, most recently from f5bb6a0 to ac62e20 Compare January 10, 2025 19:55
@AnujChhikara
Copy link
Contributor

LGTM

@iamitprakash iamitprakash merged commit c4263c9 into Real-Dev-Squad:develop Jan 15, 2025
3 checks passed
@Achintya-Chatterjee Achintya-Chatterjee mentioned this pull request Jan 15, 2025
10 tasks
yesyash pushed a commit that referenced this pull request Jan 18, 2025
* fix: Add support to fetch onboarding extension by requestedBy field
- Add requestedBy field in the fetch query
- Add tests to verify the chanes

* fix: Add changes under dev flag and fix failing tests

* chore: fix typo
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.

Feature : Extend Onboarding Override for Discord Server
5 participants