News-At-Command-Line icon indicating copy to clipboard operation
News-At-Command-Line copied to clipboard

Adding options to open the saved articles.

Open Griffintaur opened this issue 7 years ago • 8 comments

Saving the article feature has already been introduced and now the feature for letting the user view its saved articles needs to be taken up.

Griffintaur avatar Nov 16 '17 03:11 Griffintaur

Hi!

I would like to participate in this, I am new at contributing to open source, so if you think it is not a good start for me, please let me know and I can take on other first issues.

-Andres

drusc0 avatar Nov 19 '17 21:11 drusc0

yes, you can go for it.

Griffintaur avatar Nov 20 '17 03:11 Griffintaur

or you can check my new project ( https://github.com/Griffintaur/Padestrian-Detection) where you could work on the issue and also list other issues if you find any.

Thanks and Regards, Ankit Singh

On Mon, Nov 20, 2017 at 5:32 AM, drusc0 [email protected] wrote:

Hi!

I would like to participate in this, I am new at contributing to open source, so if you think it is not a good start for me, please let me know and I can take on other first issues.

-Andres

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Griffintaur/News-At-Command-Line/issues/22#issuecomment-345551393, or mute the thread https://github.com/notifications/unsubscribe-auth/AKD_RWIsMEjJ4mTw5EfDoSiUPKZoVgIyks5s4J5VgaJpZM4Qf6Ra .

Griffintaur avatar Nov 20 '17 05:11 Griffintaur

ok cool, I will check that one out too. But for this issue, what I was thinking was to add another option (like 99 to quit, maybe 88 to look at what was saved) in the main script, or if you have some other thing in mind, share with me and we can come to a conclusion.

Thanks. -Andres

drusc0 avatar Nov 22 '17 05:11 drusc0

I think it is better to have word based commands rather than number based.

for e.g press (q) to quit, S (to save) and O to open.

I think these things are common in terminal-based programs. Do you have any other idea?

On Wed, Nov 22, 2017 at 1:12 PM, drusc0 [email protected] wrote:

ok cool, I will check that one out too. But for this issue, what I was thinking was to add another option (like 99 to quit, maybe 88 to look at what was saved) in the main script, or if you have some other thing in mind, share with me and we can come to a conclusion.

Thanks. -Andres

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Griffintaur/News-At-Command-Line/issues/22#issuecomment-346245590, or mute the thread https://github.com/notifications/unsubscribe-auth/AKD_RS2TvzBXafTlkv_mLSK7yaXvTBn4ks5s461FgaJpZM4Qf6Ra .

Griffintaur avatar Nov 23 '17 03:11 Griffintaur

not at the moment. I will work on this and in case I get some, I will update the case.

drusc0 avatar Nov 28 '17 02:11 drusc0

Should the "saved files" then be another option in the configuration file? right now the interaction consists of numbers which is the source, and 99 to quit. Also, as I was playing with it, I noticed that the saving only creates a file, but it does not store any data.

drusc0 avatar Nov 28 '17 03:11 drusc0

Hey I am new to Open Source,can some one tell me,why I am recieving this type of error,when selecting a particular article.. HTMLParser.HTMLParseError: malformed start tag, at line 177, column 65 Didn't you face the same issue?

NitishDobriyal avatar Jan 27 '18 13:01 NitishDobriyal