Go to TIC4002 main site
  • Dashboards home
  • Participation dashboard
  • Forum dashboard
    • **iP dashboards**
  • iP progress
  • iP comments
  • iP Code
    •
    **tP dashboards**
  • tP progress
  • tP comments
  • tP Code
    •

    tP comments dashboard

    Sorted based on the number of comments given to others' PRs, but also showing comments on own PRs and other comments given. Updated weekly.

    [This page was last updated on Mar 25 2021]

    Same as the comment above.

    Perhaps the second NRIC field should be removed?

    Line 190: Perhaps the NRIC field could also be shifted to after Name?

    Perhaps S1111112B could be replaced with S1234567B so that it's easier to check the correct format?

    S12345678A has 8 numbers instead of the required 7.

    These extra newlines should be removed.

    There are two date fields.

    Same case as above.

    The example add dat is not in the second position.

    Edit and Find date field is not in the second position.

    There might be an issue with this.

    Previously it was getName AND (getNric OR getPhone).

    Now it is getName AND getNric AND getPhone AND getEmail.

    This should also check for leap years, it would be 29 days instead.

    Line 43 and 44 are the same.

    I have added the remark command, field and tests in this commit.

    Pushing from wrong branch.

    Format: followup INDEX NUMBER_OF_DAYS

    Show exclamation icon when person to be called is today

    Show number of days set for the call interval

    Perhaps this should be list all employees in Employee Tracker instead to make it more consistent with our naming conventions?

    Perhaps Employee Tracker is more consistent with our naming convention than address book?

    Perhaps [n/NAME] should be removed?

    Perhaps [r/ROLE] should be removed?

    Does not add value to both users and developers who are maintaining the code

    ok

    done

    done

    removed the duplicated one

    corrected

    done

    Finished Developers guide

    For tutorial learning

    Done

    Thanks!

    Good

    done

    close #4, close #8, close #13

    The PRs created for tutorials need not be merged.

    @e0260222 Hi Weizhong Please help to add Milestone and merge it. thanks

    Please help merge. thanks @e0260222

    @e0260222 Hi Weizhong, Please help to add milestone and merge it. thanks

    @e0260222 Please help merge it. thanks

    Hi @e0260222 , Please help merge it . thanks

    no need to create PR to Team Repo for this tutorial-removing-field

    resolved merge conflict that was blocking CI

    invalid user story

    unable to replicate after merging multiple PRs was done.

    closing issue.

    closing due to incorrect branch

    Merge code, add new feature for appointment

    View appointment function is added to the app

    Now can view appointment base on seach of the patient name

    Reviewed. All good so far.

    Page is updated

    Page is updated

    View Patient data function has been updated

    Functionality exists

    Functionality exists

    Done

    Done

    task done.

    Done

    Done

    Done

    verified.

    okay

    noted

    okay. i will update it and the IC code change to NRIC and the space between IC and phone

    but cannot show in Address App

    pls, ignore the commits as it is wrong reference to this issue. it supposes be issue add NRIC.

    solve the issue regarding the show in Address app.

    updated

    done

    done

    updated

    done

    pls help me review regarding update UG with NRIC

    Hi, I help you to close the issue

    Hi, I help you to close the issue

    UML(addCommandSequence diagram)

    Close due to more updates

    Change to: