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

Setup ModelAdmin class for each model in Challenge app #72

Closed
taranjeet opened this issue Nov 19, 2016 · 8 comments
Closed

Setup ModelAdmin class for each model in Challenge app #72

taranjeet opened this issue Nov 19, 2016 · 8 comments

Comments

@taranjeet
Copy link
Member

This class should be derived from a TimeStampedAdmin of base app.

Notable attributes should be

  • list_display
  • search_fields
  • list_filter

Note that this issue depends on Issue 68 which involves the creation of TimeStampedAdmin class.

@gauthamzz
Copy link
Contributor

can i try this ?

@taranjeet
Copy link
Member Author

@gauthamzz : yeah sure, but if you dont mind can you give #68 a try. This is because TimeStampedAdmin class does not exist as of now and we need it here.

@gauthamzz
Copy link
Contributor

Can you give me an idea how to solve #68

@taranjeet
Copy link
Member Author

@gauthamzz : as of now you can start by just writing a modelAdmin class. This class will register the model on django admin site.

There are certain attributes for this class, which defines how can you search on this model, what filters will be available and so on. We are only concerned with three attributes

@Arun-Jain
Copy link
Contributor

can i try this?

@gauthamzz
Copy link
Contributor

i am almost done with this , try other part

@Arun-Jain
Copy link
Contributor

Arun-Jain commented Nov 23, 2016

@gauthamzz Sure

@deshraj
Copy link
Member

deshraj commented Dec 3, 2016

Fixed in #149

@deshraj deshraj closed this as completed Dec 3, 2016
deshraj pushed a commit that referenced this issue May 31, 2020
deshraj pushed a commit that referenced this issue Oct 14, 2020
…#92)

* Fix token reset while running tests

* Fix os path impor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants