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

Backend Connection, Lazy Loading, Single Post View #8

Merged
merged 24 commits into from
Jul 2, 2018
Merged

Backend Connection, Lazy Loading, Single Post View #8

merged 24 commits into from
Jul 2, 2018

Conversation

ateufel
Copy link
Contributor

@ateufel ateufel commented Jul 1, 2018

  • config.js for all the current and upcoming basic settings (Steem Connect scope array, for example).
  • Implement API handler/functions connecting frontend to backend.
  • Push new contributions not only to the blockchain, but to the backend too.
  • Load user articles from backend instead of using mock data.
  • Load articles by category from backend instead of using mock data.
  • Implement Lazy Loading for all article lists.
  • Single post view: show title, content, tags, ...
  • Convert timestamp to pretty human readable date.
  • Add some spinners/loaderes for article list and article detail page.
  • Update all npm packages, fix breaking changes of ESLint 5.x.

ateufel added 23 commits June 23, 2018 22:48
…ons. add loader for posting and redirect to contributions overview.
…c article detail page with backend data. fix routing for detail page.
@ateufel
Copy link
Contributor Author

ateufel commented Jul 1, 2018

commits from PR 7 show up as well, sorry for that. PR 7 commits are from the beginning to (and including) "remove redundant state variable change" (#7)

.eslintrc Outdated
@@ -9,11 +9,10 @@

//http://eslint.org/docs/rules/
"parserOptions": {
"ecmaVersion": 8,
"ecmaVersion": 2018,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please fix this indentation

Copy link
Member

@ms10398 ms10398 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only one little fix and we are good to go

Try to rebase your branch after it avoid showing of previous commits

@onozasi onozasi merged commit 3aa125d into knacksteem:master Jul 2, 2018
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