-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add META SUMMARY etc. to app market #51
Comments
See also opengraph metadata, e.g. https://blog.kissmetrics.com/open-graph-meta-tags/ |
FlowRouter has support for server-side-rendering, but only if you're using ReactLayout: https://github.com/kadirahq/flow-router/tree/ssr We'd probably also have to do something about the fact that the app market server doesn't actually have a copy of the app index data to work with, which means it can't really render anything useful. So there'd be some work to integrate fetching (and caching?) the appropriate app index data serverside. |
In retrospect, perhaps doing the app market as a Meteor app (rather than static content) was a mistake. |
Yeah - I think for this project, if we had gotten the feature set we had
hoped for, then dynamic would have been essential. We do still have the
ratings, though.
But anyway.
|
Rationale:
Details:
The text was updated successfully, but these errors were encountered: