Skip to content
This repository has been archived by the owner on Jan 31, 2018. It is now read-only.

Admin pages 788527 #22

Closed
wants to merge 2 commits into from
Closed

Admin pages 788527 #22

wants to merge 2 commits into from

Conversation

willkg
Copy link
Member

@willkg willkg commented Sep 6, 2012

This adds admin and adminplus infrastructure.

It also adds a basic index infrastructure page that shows the status of things.

Indexing from the admin will come in a future pull request.

r?

This version of the search index admin doesn't do much other than tell
you what's going on index/mapping-type wise in your system.
@@ -0,0 +1,103 @@
{% extends "admin/base_site.html" %}
Copy link
Member Author

Choose a reason for hiding this comment

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

As a side note, the admin stuff is Django templates.

@mythmon
Copy link
Contributor

mythmon commented Sep 6, 2012

Some minor issues, but the major details look right to me. Maybe @rehandalal has an opinion about the HTML/CSS stuff?

@willkg
Copy link
Member Author

willkg commented Sep 6, 2012

Because this is part of the django admin stuff, the html/css stuff is pretty limited to what I can hack in. That's one of the reasons it's coded the way it is.

@mythmon
Copy link
Contributor

mythmon commented Sep 7, 2012

Ah, right. I didn't realize that the Django admin would make styling annoying. In that case the html/css is fine. The indentation isn't a big issue either, it just looks weird to me.

@willkg
Copy link
Member Author

willkg commented Sep 7, 2012

Made fixes @mythmon suggested and landed in 80e9cad

@willkg willkg closed this Sep 7, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants