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

Updated flowers and info command in UserGuide.md #99

Merged
merged 1 commit into from
Apr 3, 2024
Merged

Conversation

samuelory
Copy link

No description provided.

@samuelory samuelory added this to the v2.0 milestone Apr 3, 2024
@samuelory samuelory added the priority.Medium Nice to have label Apr 3, 2024
@samuelory samuelory self-assigned this Apr 3, 2024
Copy link

@IanFH IanFH left a comment

Choose a reason for hiding this comment

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

UG looks good to me

Copy link

@JeffinsonDarmawan JeffinsonDarmawan left a comment

Choose a reason for hiding this comment

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

Good addition of userguide for flowers

Comment on lines +77 to +93
Format: `flowers <occassion>`

Example: `flowers`

Expected output:
```
Showing page 1/2 of all the flowers you can add:
1. White Orchid
2. Dark crimson Rose
3. Red Rose
4. White Lily
5. White Daisy
Type 'next' to go to the next page.
____________________________________________________________
```

Example: `flowers Funeral`
Copy link

Choose a reason for hiding this comment

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

Very detailed, good job!

Comment on lines +105 to +108
Get detailed info (colour, meaning and related occasions) about a specific flower in the database,
will show all colours and their associated meanings (specifying colours will come soon).

Shows at most five flowers at once, type `next` to see flowers on the next page, and `back` to view a previous page.
Copy link

Choose a reason for hiding this comment

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

Clear instructions well done!

@samuelory samuelory merged commit e6bab99 into master Apr 3, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority.Medium Nice to have
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants