Skip to content

added saved articles to be displayed if present, with possibility to …#23

Open
drusc0 wants to merge 1 commit into
Griffintaur:masterfrom
drusc0:saved-articles
Open

added saved articles to be displayed if present, with possibility to …#23
drusc0 wants to merge 1 commit into
Griffintaur:masterfrom
drusc0:saved-articles

Conversation

@drusc0

@drusc0 drusc0 commented Dec 8, 2017

Copy link
Copy Markdown

…delete them after reading them. changed the format in which the files are saved to json

…delete them after reading them. changed the format in which the files are saved to json
@Griffintaur

Copy link
Copy Markdown
Owner

@drusc0 any reason behind going for json?
or just because you want to store title and its corresponding url?
are you saving the articles into separate folder

@drusc0

drusc0 commented Feb 22, 2018

Copy link
Copy Markdown
Author

@Griffintaur no reason at all. But for retrieving the file and reporting back on terminal, it was easier to have a key-value pair. Is there any other format or you prefer text to cat on screen? I can make the change.

@Griffintaur

Copy link
Copy Markdown
Owner

Please resolve the merge conflict @drusc0

@drusc0

drusc0 commented Oct 6, 2018

Copy link
Copy Markdown
Author

are those changes needed, @Griffintaur ? I thought someone had made other changes... let me know if this is necessary and ill fix otherwise i will cancel the pull request.

thanks.

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