-
Notifications
You must be signed in to change notification settings - Fork 386
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
move main pages from v4.loopback.io #840
Conversation
f380533
to
ee58cc8
Compare
@@ -0,0 +1,202 @@ | |||
--- |
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.
rename it from index.html
.
index.html
Outdated
|
||
<!--Header alert --> | ||
<div class="header-alert"> | ||
<p class="centered-regular">LoopBack 4 GA (General Availability) has been released in October 2018, read more in <a href="http://strongloop.com/strongblog/loopback-4-ga" target="_blank">the announcement post.</a> Click <a href="/lb3">here</a> for LoopBack 3 pages.</p> |
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.
I've edited the alert to show how to access the LB3 pages. Not sure if it's necessary.
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.
I am not too familiar with the changes, but from the demo you gave the team and the direction this PR is taking, LGTM. It makes sense to have the landing page of loopback.io
point to the current release of our framework 💪.
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.
Two things:
- favicon is still the green one for the v4 pages
- clicking on the LoopBack symbol on the green
/lb3
pages leads back to the main lb4 page
Other than that LGTM 👍
It seems to be working for me, i.e. showing the LB4 logo as the favicon. Do you mind opening the browser in incognito mode to try it out again? Thanks.
I think it makes sense to bring back to the main page loopback.io when clicking on the old LB logo? |
Tried it on Firefox and it's the right one 👍
How would you go back to the main lb3 page? I'm fine with how it is now, but just wondering though. |
I think it will go back to the same way how it brings you to the /lb3. An alternative is to put "LoopBack 3" in the navbar, which may not be a bad idea. |
@nabdelgadir, I think i like this one better.
WDYT? |
@dhmlau LGTM 👍 |
@slnode test please |
@dhmlau Some quick feedback:
|
I see this error from the console:
|
I noticed that too. but couldn't find this jpg anywhere, I'll remove the reference of it to get rid of the error.
|
I suggest that we get rid of LB4 GA banner. It’s not adding much value any more. |
What about a shortcut for blogs? We are good at publishing blogs and they are very useful information? |
@dhmlau The link for |
Sorry about that @raymondfeng @nabdelgadir. I did some more testing and found more issues. :( |
@slnode test please |
Move the main pages from v4.loopback.io, so that the loopback.io will be showing the LB4 content.
User Experience
When user going to loopback.io, it shows the same content as v4.loopback.io. All the pages accessible from the navbar (at the top) will be the new LB4 content.
Docs
at the nav bar will be pointing to/doc/lb4
. The only way to get to LB3 documentation is either:LoopBack
, that will bring you to the list of LB versionsAfter this PR has merged, we can redirect v4.loopback.io to loopback.io.
Discussion
@strongloop/loopback-maintainers, there are a few topics that I'd like to get your inputs:
Github stars shown on loopback.io
I've changed it to show the github star for
loopback-next
repo. As you might aware already, it's a much lower number. I think it makes sense to use the LB4 repo github star even though it's less impressive at this moment.LoopBack 3 pages
For the following LB3 pages, I've "relocated" it to /lb3/* url.
i.e. loopback.io/getting-started -> loopback.io/lb3/getting-started
If we still want users to access those pages, I'd like to propose to have the banner at the top adding to say if you want to access the LoopBack 3 pages, go to
/lb3/
link. But i'm not sure if it's necessary.