Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

[6059] - rename apply api to recruits #3618

Merged
merged 2 commits into from
Sep 25, 2023
Merged

[6059] - rename apply api to recruits #3618

merged 2 commits into from
Sep 25, 2023

Conversation

d-a-v-e
Copy link
Contributor

@d-a-v-e d-a-v-e commented Sep 25, 2023

Context

https://trello.com/c/DFSHx7Y1/6059-rename-apply-api-to-recruits-api

we consume an api from "apply" but it shouldn't be called that, we've opted for "recruits"

Changes proposed in this pull request

renames all modules and references from apply to recruits

@d-a-v-e
Copy link
Contributor Author

d-a-v-e commented Sep 25, 2023

Rename "Apply API" to "Recruits API"

carlosmartinez
carlosmartinez approved these changes Sep 25, 2023
View reviewed changes
Copy link
Collaborator

@carlosmartinez carlosmartinez left a comment

Choose a reason for hiding this comment

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

@sonarcloud
Copy link

sonarcloud bot commented Sep 25, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

darokel
darokel approved these changes Sep 25, 2023
View reviewed changes
Copy link
Contributor

@darokel darokel left a comment

Choose a reason for hiding this comment

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

🥇

@d-a-v-e d-a-v-e merged commit ba03332 into main Sep 25, 2023
32 checks passed
@d-a-v-e d-a-v-e deleted the 6059-rename-apply-api-to-recruits-api branch September 25, 2023 12:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@darokel darokel darokel approved these changes

@carlosmartinez carlosmartinez carlosmartinez approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@d-a-v-e @carlosmartinez @darokel