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

Creating a Sequence Diagram for Searching an Article by Tags #52

Closed
burakmert236 opened this issue Apr 5, 2022 · 5 comments
Closed
Assignees
Labels
Prio: Medium Importance of Issue: Medium Type: Documentation Improvements or additions to documentation

Comments

@burakmert236
Copy link
Contributor

burakmert236 commented Apr 5, 2022

Description: Creating a Sequence Diagram of a use case for searching an article in the application. This search is performed by tags.

Reviewer: @mehmetemreakbulut
Review Deadline: 10.04.2022 15.00
Task Deadline: 10.04.2022 12.00

@burakmert236 burakmert236 added Type: Documentation Improvements or additions to documentation Prio: Medium Importance of Issue: Medium Status: In Progress Issue is worked on, currently labels Apr 5, 2022
@burakmert236 burakmert236 changed the title Creating a Sequence Diagram for Searching an Article Creating a Sequence Diagram for Searching an Article by Tags Apr 8, 2022
@burakmert236
Copy link
Contributor Author

I uploaded first version of the diagram in our team's lucidchart project. You can view this sequence diagram. You may modify it or make a comment about it.

@yavuuzsameet
Copy link
Contributor

I think we can add the 'return' keyword to dashed lines. return searchEngine, return searchHistory, etc.

@yavuuzsameet
Copy link
Contributor

After the meeting held yesterday, and a little additional research I found out that no need for 'return' keyword. Hereby, I withdraw my request to add the 'return' keyword to the diagram. It is now good as it is. Thanks.

@mehmetemreakbulut
Copy link
Contributor

I think the diagram is done successfully. For more suggestions, we can make the final review after the customer's return.

@burakmert236 burakmert236 removed the Status: In Progress Issue is worked on, currently label Apr 18, 2022
@burakmert236
Copy link
Contributor Author

We updated the diagram according to customer's review and now I'am closing the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Prio: Medium Importance of Issue: Medium Type: Documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

4 participants