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

[T16-3] FoodZoom #124

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

Conversation

kohchihao
Copy link

@kohchihao kohchihao commented Oct 11, 2018

Previously closed PR : #8
@juxd @rrtheonlyone @monmanuela @jinyingtan

Copy link
Member

@tanhengyeow tanhengyeow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello all! To address your query on why Recommended progress for the upcoming milestone isn't flagging anything, please take note of the following:

  1. Your team should change labels to match the original ones given, otherwise the bot cannot detect them! Please fix the label naming by next week :) Don't worry about this week's labels.
  2. The bot team would ignore case insensitivity from now on, so it should be fine! But if you want to be safe, change the milestones to lowercase.

Copy link
Member

@tanhengyeow tanhengyeow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work so far! As mentioned in tutorial, do prioritize the remaining tasks and break them down into manageable chunks for the rest of the milestones 😄

Ideally, all the major tasks should be done by v1.3 so the team can focus on fixing bugs, docs and handling minor updates.

@kohchihao
Copy link
Author

@tanhengyeow We have all finished our part on the Implementation inside DeveloperGuide. Please review 😄

Jin Ying and others added 30 commits November 12, 2018 10:33
Fix wrong messages shown on edit, assign completed order and allow clear order if no ongoing orders
Add lines in asciidoctor.css to fix links in pdf
Fix find command bugs for order and deliveryman
* Tweaks to PPP

* Add new line at EOF

* Change word to plural
DeveloperGuide.adoc: Add deliveryman find for AppendixH
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants