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

Add RawGenericKey widget #14

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

dmclain
Copy link
Member

@dmclain dmclain commented Mar 7, 2012

The visual search based generickey widget can work with a lot of additional work as seen in the WellNodeInline, but it doesn't provide the easy searching/filtering that the django raw picker does. The RawGenericKey widget provides an interim solution that uses django's existing raw foreign key picker in combination with some javascript glue to make it easy to pick an arbitrary object out of the system.

@tswicegood
Copy link
Contributor

I'd really like to see some splinter tests for this. Any chance you want to take a stab at it? Considering that the admin is one of the biggest parts of Armstrong, it's pretty concerning that there's no test coverage for it.

@niran
Copy link
Contributor

niran commented Mar 19, 2012

The likelihood of getting this tested with Splinter before the end of the month seems low. Let's release the functionality then add tests in early April.

class RawGenericKeyWidget(Widget):
template = "admin/hatband/widgets/rawgenerickey.html"

class Media:
Copy link
Contributor

Choose a reason for hiding this comment

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

This should respect ARMSTRONG_ADMIN_PROVIDE_STATIC (see 905d2ae)

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.

3 participants