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

Make a landing page at /about #296

Closed
Martii opened this issue Aug 18, 2014 · 2 comments · Fixed by #328
Closed

Make a landing page at /about #296

Martii opened this issue Aug 18, 2014 · 2 comments · Fixed by #328
Labels
DOC Pertains inclusively to the Documentation operations. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. needs discussion Blah, blah, blah, wahh, wahh, wahh, etc. team biz This is similar to a meta discussion.

Comments

@Martii
Copy link
Member

Martii commented Aug 18, 2014

This should show some basic intel:

  • Welcome message although could use a mission statement or something touting our site.
  • Link to /about/about, or move that into /about, or an actual page TOC?
  • Make controller synchronous for pulling the pages list Needs verification on code structure
  • Add prerender processor to handle header bookmarks for static /about page or just hardcode.
  • What node version is running if possible
  • What app version is running if possible
  • Perhaps $ npm list if possible Applied at Add npm list --json to /admin #318 in a unique admin route
  • GitHub deploy commit hash to see if we have a particular HEAD deployed... so we can stop asking "Is it deployed yet?"
  • Move anything else we want/can into that page... perhaps the Collaborators link and remove from footer? n/m on the footer

Any other ideas? Assigning to self for the time being although may be slower due to BLOCKING flag.

@Martii Martii self-assigned this Aug 18, 2014
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 19, 2014
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 19, 2014
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 19, 2014
@jerone
Copy link
Contributor

jerone commented Aug 19, 2014

  • GitHub deploy commit hash to see if we have a particular HEAD deployed... so we can stop asking "Is it deployed yet?"

👍

  • What node version is running if possible
  • Perhaps $ npm list if possible

Isn't this security sensitive... Could be moved to the admin page.

@Martii
Copy link
Member Author

Martii commented Aug 19, 2014

Isn't this security sensitive... Could be moved to the admin page.

Perhaps... that's why I'm asking for team input. I do like the commit hash being public though. That way administration can be "look here" in case someone is wondering.

I'm still twiddling with the async package at the moment getting this pseudo synchronous because 1 time out of 1000 the Pages list was blank. heh... so learning a new technology for node. :)... and since no one answered me months ago about what waterfall was... my learning curve on this is somewhat improved. :)

Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 19, 2014
Applies to OpenUserJS#296

**NOTE**: This could use some verification to ensure it is being coded correctly. Works with standard testing on dev.
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 19, 2014
* Set currently to text since that's what those are

Applies to OpenUserJS#296
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 19, 2014
Applies to OpenUserJS#296

**NOTE**: This doesn't appear to make a huge difference either way but symmetrical with null callback syntax with the implied return
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 19, 2014
Applies to OpenUserJS#296

**NOTE** Prior code deduced from other routines which probably have the same issue.
Martii referenced this issue Aug 19, 2014
* Add hardcoded bookmarks
* Add Mission header with tba
* Add Getting Started header
* Show node version for admins only
Martii referenced this issue Aug 19, 2014
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 20, 2014
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 23, 2014
* Tack on a virtual model name since we have no control over this implied native DB model.

Applies to OpenUserJS#296
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 23, 2014
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 23, 2014
* Show the raw npm ls JSON data for the server

Applies to OpenUserJS#296

This appears to be the only way it can be done for listing the current dependencies. I haven't found any native API to Node.js that does this... so I'm willing to give this a whirl.
Martii pushed a commit to Martii/OpenUserJS.org that referenced this issue Aug 25, 2014
Closes OpenUserJS#296

Feature for GH commit hash added will need to be done somewhere in deployment if possible so that feature needs to be down the line.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
DOC Pertains inclusively to the Documentation operations. enhancement Something we do have implemented already but needs improvement upon to the best of knowledge. needs discussion Blah, blah, blah, wahh, wahh, wahh, etc. team biz This is similar to a meta discussion.
Development

Successfully merging a pull request may close this issue.

2 participants