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

Event Management #61

Merged
merged 32 commits into from
Sep 25, 2018
Merged

Event Management #61

merged 32 commits into from
Sep 25, 2018

Conversation

Kratious
Copy link

No description provided.

@Kratious
Add Event class and associated parameter classes
acf9d7d
@Kratious
Add tests for Event Name and Date classes
11d65cb
@Kratious
Add Event operations to AddressBook
15a9a93
@Kratious
Add Event operations to ModelManager.java and Model.java
b807085
@Kratious
Add file path for eventlist.xml in UserPrefs.java
4ad9b13
@Kratious
Update Event class and associated Id class
fa81184
@Kratious
Add getEventListFilePath() method to Storage.java and AddressBookStor…
f0953e4 
…age.java
@Kratious
Add XmlAdaptedEvent.java
a714c6a
@Kratious
Merge branch 'master' of https://github.com/CS2103-AY1819S1-W16-2/main …
f6ef1a3 
…into event-class
@Kratious
Updated Description.java to take in any values instead of just alphan…
e30bdd6 
…umeric
@Kratious
Add sample events for SampleDataUtil.java and getSampleAddressBook() …
619e949 
…to generate sample events
@Kratious
Rectify delimiter error from '-' to ':' in Time.java
15579d8
@Kratious
Revert eventListFilePath in AddressBookStorage.java and Storage.java
e5b053a
@Kratious
Add list of Event objects to XmlSerializableAddressBook.java
b2ed54b
@Kratious Kratious added this to the v1.1 milestone Sep 24, 2018
@coveralls
Copy link

coveralls commented Sep 24, 2018 •
edited
Loading

Coverage Status

Coverage decreased (-1.4%) to 91.393% when pulling 4104d14 on Kratious:event-class into e0345f0 on CS2103-AY1819S1-W16-2:master.

@iMarbles iMarbles added this to To do in To-Do Tracker via automation 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
@Kratious Kratious moved this from To do to In progress in To-Do Tracker Sep 24, 2018
@iMarbles iMarbles changed the title Event class implemented. Initial storage of sample events with SampleDataUtil working. Event Management Sep 24, 2018
@Kratious
Copy link
Author

Should abstract hasPerson, addPerson, updatePerson, removePerson methods in ModelManager.java and AddressBook.java to support the three different entities (Volunteers, Events, Records)

@iMarbles iMarbles removed the status.Ongoing The issue is currently being worked on. note: remove this label before closing an issue. label Sep 25, 2018
@iMarbles iMarbles merged commit f08ca4f into CS2103-AY1819S1-W16-2:master Sep 25, 2018
To-Do Tracker automation moved this from In progress to Done Sep 25, 2018
@Kratious Kratious deleted the event-class branch September 25, 2018 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iMarbles iMarbles iMarbles left review comments

@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
3 participants
@Kratious @coveralls @iMarbles