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

fix(issue#185): add my list for in list command #224

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jufab
Copy link
Contributor

@jufab jufab commented Oct 24, 2023

Summary

This is a proposal for issue #185

There is incoming list and todo list

image

and

image

Ticket Link

fixes #185

@jufab jufab requested a review from larkox as a code owner October 24, 2023 20:46
@larkox larkox requested a review from mickmister October 25, 2023 08:37
Copy link
Contributor

@mickmister mickmister left a comment

Choose a reason for hiding this comment

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

LGTM 👍 Thanks @jufab!

@mickmister mickmister added the 3: QA Review Requires review by a QA tester label Oct 25, 2023
@avas27JTG avas27JTG added this to the v1.0.0 milestone Dec 22, 2023
@AayushChaudhary0001
Copy link

@jufab I have tested this PR and it does not looks like working the way it is shown. On running /todo list it shows empty even after the user has something in accepted Todo's and Incoming Todo's

image

On running /todo list

image

@AayushChaudhary0001
Copy link

@jufab A gentle reminder for this.

@raghavaggarwal2308
Copy link
Contributor

@jufab Gentle reminder for this

1 similar comment
@raghavaggarwal2308
Copy link
Contributor

@jufab Gentle reminder for this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3: QA Review Requires review by a QA tester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The ToDo lint "in" behavior is unclear
6 participants