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

Handle empty response from server in Loklak Fetcher #881

Merged
merged 1 commit into from
Jan 9, 2017

Conversation

hoangvanthien
Copy link
Member

@hoangvanthien hoangvanthien commented Jan 9, 2017

Thank you for your pull request! Please read and understand everything below

Please acknowledge that you have read and understood each of the following before submitting your pull request.

Do not delete any text other than where you are instructed.

Maintainers: If any of these are left unchecked, do not merge.

Students: If one of them is not applicable to you -- Please check it anyways. DO NOT REMOVE ANYTHING!

You may check them after opening your pull request.

You may do so by changing each [ ] to [x] Please take note of the whitespace as it matters.

  • Read and understood (see CONTRIBUTING.md)
  • Included a live link or preview screenshot
  • Images are 240 x 240 [w x h]
  • Resolved merge conflicts
  • Included a description of my change below

Things done in this Pull Request

I encounter an error of empty response issued in #880 (I made it strikethrough since it doesn't seem an error of our website).

However, in this case, I strongly suggest that we must halt the parsing function instead of continuing with empty object. See more in file changes.

@checkbot
Copy link

checkbot commented Jan 9, 2017

1 Warning
⚠️ Please mark all the checkboxes.
1 Message
📖 Hi @hoangvanthien thank you for your submission at Fossasia’s GCI 2016 website. Please correct any issues above, if any.

🔗 Maintainers: @hoangvanthien did not provide a live link, one can be found at https://hoangvanthien.github.io/gci16.fossasia.org

Generated by 🚫 danger

@robbyoconnor robbyoconnor merged commit ca225c1 into fossasia:gh-pages Jan 9, 2017
skywrapper pushed a commit to skywrapper/gci16.fossasia.org that referenced this pull request Jan 10, 2017
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.

3 participants