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

[CS2103T-W13-1] HrManager #96

Open
wants to merge 794 commits into
base: master
Choose a base branch
from

Conversation

sw-boi
Copy link

@sw-boi sw-boi commented Sep 16, 2021

HR Manager will help you manage the candidates to be interviewed,
making the scheduling process easier and faster for your company!

Tired of losing track of scheduled interviews?

HR manager's easy to use features will help you to arrange for upcoming interviews quickly in your desired manner.

The data you have provided will also be stored safely and securely for subsequent uses,
transferable to other devices too!

@codecov-commenter
Copy link

codecov-commenter commented Sep 26, 2021

Codecov Report

Merging #96 (989b09b) into master (ba53b8c) will increase coverage by 5.91%.
The diff coverage is 81.80%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #96      +/-   ##
============================================
+ Coverage     72.15%   78.07%   +5.91%     
- Complexity      399      993     +594     
============================================
  Files            70      119      +49     
  Lines          1232     3033    +1801     
  Branches        125      417     +292     
============================================
+ Hits            889     2368    +1479     
- Misses          311      540     +229     
- Partials         32      125      +93     
Impacted Files Coverage Δ
src/main/java/seedu/address/MainApp.java 0.00% <0.00%> (ø)
...main/java/seedu/address/commons/core/Messages.java 0.00% <ø> (ø)
src/main/java/seedu/address/logic/Command.java 100.00% <ø> (ø)
...s/logic/candidate/exceptions/CommandException.java 100.00% <ø> (ø)
...c/main/java/seedu/address/logic/parser/Prefix.java 91.66% <ø> (ø)
.../main/java/seedu/address/model/person/Address.java 90.00% <ø> (+10.00%) ⬆️
...rc/main/java/seedu/address/model/person/Email.java 90.00% <ø> (+10.00%) ⬆️
src/main/java/seedu/address/model/person/Name.java 90.00% <ø> (+10.00%) ⬆️
...rc/main/java/seedu/address/model/person/Phone.java 90.00% <ø> (+10.00%) ⬆️
...a/seedu/address/model/person/UniquePersonList.java 92.85% <0.00%> (+0.17%) ⬆️
... and 99 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ba53b8c...989b09b. Read the comment docs.

bentanjunrong pushed a commit to bentanjunrong/tp that referenced this pull request Oct 18, 2021
@sw-boi sw-boi changed the title [CS2103T-W13-1] ExpenseManager [CS2103T-W13-1] HrManager Oct 26, 2021
okyntary pushed a commit to okyntary/tp that referenced this pull request Oct 26, 2021
sueann-chua and others added 26 commits November 4, 2021 15:21
Update ug interview and position section
* moved table of inputs for interview related commands to the beginning of Interview Management section
Update UserGuide, Constraint Message for Duration in Interview and AddInterviewCommandParser
…into remove-edit_i-candidates_functionality

# Conflicts:
#	docs/UserGuide.md
* instead of wrapping individual labels for each interview, create a single label with all interviews and then wrap
* added "wrapText=true" to most fields
* removed unnecessary setWrapText method in PersonCard
* added "wrapText=true" for name field in fxml file
* Fixed remark candidateCommand for bugs related to Interview (credits to seaweediman)
* added a for loop to ensure maximum string length per line does not exceed 60
* fixed Checkstyle error
sw-boi and others added 30 commits November 8, 2021 20:06
Add lines to separate sections for readability
Add back enhancements for PPP
Update DG, FindXCommand, Assign and Unaasign
* changed description of edit_c to match the behaviour of the application
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.

6 participants