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

Add text 'No answers for this question ...' to avoid simple exit #47

Merged
merged 1 commit into from
Apr 21, 2017

Conversation

arount
Copy link
Contributor

@arount arount commented Apr 20, 2017

When using socli -iq and selecting an unanswered question socli just quit without any messages.

This make several issues:

  • You can not read unanswered question
  • Socli will just exit if you select an empty question (disturbing, especially in interactive mode)
  • It looks like a big bug, but it's not.

I just added a pseudo-answer in answers list if this one is empty. Like that when selecting an empty question in interactive mode I'm able to read the question and just have a message 'No answers for this question...' which is explicit.

@arount arount changed the title Add text 'No answers for this question ...' to avoid simple exist Add text 'No answers for this question ...' to avoid simple exit Apr 20, 2017
@gautamkrishnar gautamkrishnar merged commit 235a19d into gautamkrishnar:master Apr 21, 2017
@gautamkrishnar
Copy link
Owner

😃 Thanks a lot for your first contribution... You rocks!

@arount
Copy link
Contributor Author

arount commented Apr 21, 2017

\o/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants