-
Notifications
You must be signed in to change notification settings - Fork 253
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
Server-Side Rendering #1089
Server-Side Rendering #1089
Conversation
I haven't finished everything I wanted to do with SSR just yet, but everything so far works. I would like to merge those changes as quickly as possible, so we don't have two conflicting branches causing problems while merging. |
Not too familiar with server side rendering stuff, but I've been testing performances for large data sets that get rendered, like in the Activity page. And there seems to be a performance increase, well done: 👍 Compare filtering here: https://busy-nd-pr-1089.herokuapp.com/@timsaid/activity to filtering on here: https://busy.org/@timsaid/activity |
- Initialize SteemConnect - Move wildcard route to the bottom
I have a small loading icon and these scrolls when i open a profile with french locale: https://busy-nd-pr-1089.herokuapp.com/@busy.org |
I have seen this issue (scrollbars) and it was on my todo list, I will fix this shortly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really great work 👏 , it's working perfectly for me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, works fine for me!
This PR merges master into stable. 247f256 (HEAD -> master, upstream/master) Fix scroll to comment behaviour (#1083) cf1c39d Update Czech translation (#1110) 43c7735 Add Sentry's Raven for error handling (#1112) 012930e Server-Side Rendering (#1089) afed04f Show more decimals for vote percentage, add power down message, simplify activity filter search (#1104) d1fed2f Update AuthorRewardMessage to calculate vests to SP (#1103) 71cf88a New Crowdin translations (#1101)
Fixes #1023
Fixes #877
Changes:
client
andserver
insrc
folder.common
folder.fetchData
static methods.needs
code.