Skip to content
This repository has been archived by the owner on Sep 26, 2022. It is now read-only.

Adding placeholder login page (it's ugly) #53

Merged
merged 1 commit into from Mar 2, 2015

Conversation

pdehaan
Copy link
Contributor

@pdehaan pdehaan commented Feb 24, 2015

No description provided.

@pdehaan
Copy link
Contributor Author

pdehaan commented Feb 24, 2015

This is just a proof of concept, we'll replace the login logic with Persona+LDAP in a future PR.
Credentials are:
u: johnny
p: quest

@pdehaan
Copy link
Contributor Author

pdehaan commented Feb 24, 2015

Rebased!

@pdehaan
Copy link
Contributor Author

pdehaan commented Feb 24, 2015

@@ -36,6 +36,9 @@
<!-- -->
<li class="divider"></li>
<li><a ng-href="{{ ROUTES.REFERENCE }}">Reference</a></li>
<!-- -->
<li class="divider"></li>
<li><a href="/logout">Logout</a></li>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure if this /logout route should be added to the {{ ROUTES.LOGOUT }} dict like everything else. I'm rather indifferent, bordering on "meh" (but I am a fan of consistency).

@pdehaan
Copy link
Contributor Author

pdehaan commented Feb 25, 2015

r? @rpappalax @darkwing

For your coding amusement... Baby steps!
We now have the ability to hide the site behind a super-lame login page with a hardcoded username/password.

Next step (and future PR) will be to swap this out for Persona login (relevant links in #53 (comment)).

@darkwing
Copy link
Contributor

Checking this out

@darkwing
Copy link
Contributor

Works for me!

@pdehaan
Copy link
Contributor Author

pdehaan commented Feb 26, 2015

@rpappalax Did you want to re-review this before somebody presses the green button?
It's mostly the same as we glued together on Tuesday (apart from some cleaning up of routes and use of Nunjucks for templating. (I'm not married to Nunjucks, we can switch to Handlebars or Swig or something else in a future PR if you prefer.)

Once this is merged, I can ... hell, I'll work on a Convict config PR now and I can rebase it after this is merged. I need more rebase practice anyways.

@rpappalax
Copy link
Contributor

@pdehaan @kitcambridge Karl and i are going to go ahead and stand up an aws instance for loads-web on mon., and johnnyquestoauth should be OK for now, at least so david can keep working. Before we finish stitching the front and back ends together, however, we should prob have Persona integration finished. loadsv2 can spin up a large num of resources and we want to be safe.

@pdehaan
Copy link
Contributor Author

pdehaan commented Feb 26, 2015

Sounds good, sport!

refactored the hapi server routes and login template (still ugly)
@pdehaan
Copy link
Contributor Author

pdehaan commented Feb 28, 2015

rerebased.

darkwing added a commit that referenced this pull request Mar 2, 2015
Adding placeholder login page (it's ugly)
@darkwing darkwing merged commit 15efd97 into loads:master Mar 2, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants