CS2103-AY1819S1-W13-3 / main

ConTAct - A CS2103 project aimed at making a teaching assistant's job easier
https://cs2103-ay1819s1-w13-3.github.io/main/
MIT License
2 stars 5 forks source link

Add "uses" for Marks #59

Closed alyip98 closed 5 years ago

alyip98 commented 5 years ago

For other features such as attendance, edit, delete, email etc, implement the IMarkExecutable interface if the feature is supports execution with marks (e.g. batch attendance updating, batch emailing)

@preethi-d suggested format for email:

email m/MARK b/BODY

@yogtew suggested format for attendance:

attendance m/MARK at/ATTENDANCE

Suggested implementation details:

  1. Update YourCommandParser to accept a PREFIX_MARK in the tokenizer
  2. Check if argumentMultiMap contains a value for the PREFIX_MARK attribute
  3. If present, call executeMark method from execute and return the CommandResult from that
  4. Otherwise, run the rest of execute as normal

Can refer to the SetTagCommand for inspiration

alyip98 commented 5 years ago

I can write or y'all write also can haha

yogtew commented 5 years ago

what is this about?

alyip98 commented 5 years ago

Implement the interface to allow your command to be run with a Mark

alyip98 commented 5 years ago

added tag command #81

alyip98 commented 5 years ago

131