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.

Sign up for GitHub

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

Jump to bottom

Record Management: CRUD #62

Merged
merged 20 commits into from
Sep 25, 2018
Merged

Record Management: CRUD #62

merged 20 commits into from
Sep 25, 2018

Conversation

iMarbles
Copy link

@iMarbles iMarbles commented Sep 24, 2018 •
edited
Loading

Done with CRUD and tests.
Next step: Abstract UI

@iMarbles iMarbles changed the title Record Classes model/record: Record Classes Sep 24, 2018
@coveralls
Copy link

coveralls commented Sep 24, 2018 •
edited
Loading

Coverage Status

Coverage decreased (-0.5%) to 90.905% when pulling 48065f2 on iMarbles:dev into f08ca4f on CS2103-AY1819S1-W16-2:master.

@iMarbles iMarbles added this to the v1.1 milestone Sep 24, 2018
@iMarbles iMarbles added the status.Ongoing The issue is currently being worked on. note: remove this label before closing an issue. label Sep 24, 2018
@iMarbles iMarbles changed the title model/record: Record Classes Record Management Sep 24, 2018
@iMarbles iMarbles changed the title Record Management Record Management: CRUD Sep 25, 2018
@iMarbles iMarbles added status.Ready The PR is ready to be reviewed. note: remove this label before merging a PR. and removed status.Ongoing The issue is currently being worked on. note: remove this label before closing an issue. labels Sep 25, 2018
@iMarbles iMarbles added status.Ongoing The issue is currently being worked on. note: remove this label before closing an issue. status.Ready The PR is ready to be reviewed. note: remove this label before merging a PR. and removed status.Ready The PR is ready to be reviewed. note: remove this label before merging a PR. status.Ongoing The issue is currently being worked on. note: remove this label before closing an issue. labels Sep 25, 2018
@Kratious Kratious merged commit 15858fc into CS2103-AY1819S1-W16-2:master Sep 25, 2018
@iMarbles iMarbles added this to To do in To-Do Tracker via automation Sep 25, 2018
@iMarbles iMarbles removed the status.Ready The PR is ready to be reviewed. note: remove this label before merging a PR. label Sep 25, 2018
@iMarbles iMarbles moved this from To do to Done in To-Do Tracker Sep 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Scrubbius Scrubbius Awaiting requested review from Scrubbius

@sharan8 sharan8 Awaiting requested review from sharan8

Assignees
No one assigned
Labels
None yet
Projects
To-Do Tracker
  
Done
Milestone
v1.1
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@iMarbles @coveralls @Kratious @AmabelYeo