Merged
Conversation
* feat: integrate create application API * feat: add tracking button for pending application * fix: update values for create application request * test: add tests for new stepper and track application (#1143)
* feat: integrate nudge application * fix: update get application detail for normal user * fix: unit test for application detail * fix: nudge application and update nudgeCount from response * refactor: remove util and improve nudge response handling * fix: use lastNudgeAt as per api response * fix: track nudge state for reactivity
* updated newSignup flow for new users * fixed linting * fixed reivew comments * refactor: naming for role and constants * refactor: use double quotes for image cropper template --------- Co-authored-by: Mayank Bansal <mayankbansal125@gmail.com>
* feat: integrate PATCH api and edit mode for application * fix: add validation to remove unrequired fields * fix: update query param for join tests * fix: fields naming and toast messages * refactor: move socialFields to constants * test: add tests for edit application (#1148) * test: add tests for base step * test: add test for new stepper for edit mode * fix: replace college with institution
* fix: new signup redirect for staging and development * fix: add signup for test environment * fix: add login stub for redirect in test module * fix: replace window.location with ember's router
* feat: integrate image upload API * fix: modify constant for application image upload * test: add test for image upload for applications * fix: remove fileReader and use imageUrl from response * fix: update api and response for application profile upload
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
iamitprakash
approved these changes
Feb 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Date: 21-02-26
Developer Name: @MayankBansal12
Issue Ticket Number:-
PRs going for Sync-
Description:
Is Under Feature Flag
Database changes
Breaking changes (If your feature is breaking/missing something please mention pending tickets)
Is Development Tested?
Tested in staging?
Add relevant Screenshot below ( e.g test coverage etc. )
staging proofs
for user signup-
Screencast.from.2026-02-21.22-52-57.mp4
for create application-
create-proof.mp4
for edit application-
edit-proof.mp4