Removed migration script at applications/batch endpoint that added createdAt and status fields to applicants collection

[yeshavyas27]

Date: 22 August, 2024

Developer Name: @yeshavyas27

---

Issue Ticket Number

• Closes Remove Migration Script for Adding createdAt and status Fields to Applicants Collection #2094

Description

This PR removes the applications/batch endpoint and its related code that added createdAt and status fields to the applicants' collection.

Documentation Updated?

• Yes
• No

Under Feature Flag

• Yes
• No

Database Changes

• Yes
• No

Breaking Changes

• Yes
• No

Development Tested?

• Yes
• No

Test Coverage

Unit Tests:

Integration Tests:

Additional Notes

The original PR for #2094 is #2092 (closed as 2 PR for the same issue were created due to miscommunication)

[iamitprakash]

can you please add the reference of org PR

[iamitprakash]

@yeshavyas27 #2094 is still open, then what what do you mean by "Closes"

[iamitprakash]

@yeshavyas27 In desc it's says developer tested, can you add proof as well please

[yeshavyas27]

@iamitprakash
I meant that the mentioned issue will be closed if this PR is merged, indicating that PR is related to that issue. I am sorry if I used the wrong terminology, let me know how I can correct this.

I have added a screenshot under 'Test Coverage'.
By org PR you mean this: #2092 right? I will add that, also can you tell me what was the problem in that PR?

[Achintya-Chatterjee]

@yeshavyas27 the test coverage screenshot that you shared is not up to the mark. can you please update a new one, You can compare your screenshot with others for reference

[iamitprakash]

@iamitprakash I meant that the mentioned issue will be closed if this PR is merged, indicating that PR is related to that issue. I am sorry if I used the wrong terminology, let me know how I can correct this.

I have added a screenshot under 'Test Coverage'. By org PR you mean this: #2092 right? I will add that, also can you tell me what was the problem in that PR?

if this PR is in place #2092 do we really need this PR. CC- @vinit717

[iamitprakash]

@yeshavyas27 I will wait for @vinit717 to approve first as he has more context on this

[vinit717]

Make two PR's one for removing /batch route and it's code and another for removing /batch/update and it;s code and test