Skip to content

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

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

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

Conversation

@drusc0
Copy link

@drusc0 drusc0 commented Dec 8, 2017

…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
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
Copy link
Author

drusc0 commented Feb 22, 2018

@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
Owner

Please resolve the merge conflict @drusc0

@drusc0
Copy link
Author

drusc0 commented Oct 6, 2018

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