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

Orders template #1

Closed
JamesRamm opened this issue Feb 3, 2017 · 8 comments
Closed

Orders template #1

JamesRamm opened this issue Feb 3, 2017 · 8 comments
Assignees
Milestone

Comments

@JamesRamm
Copy link
Collaborator

Orders are viewable in the wagtail admin, but it is not particularly useful atm.
We could customise the admin template to allow:

  • A button to fufill orders
  • Full view of foreign key relationships (i.e. shipping address)
@JamesRamm JamesRamm self-assigned this Feb 5, 2017
@alexfromvl
Copy link
Contributor

I think the best solution is to create a dashboard, which will be the main information about the store, including the orders.

@JamesRamm
Copy link
Collaborator Author

I agree a dashboard would be a great place to eventually take it, although this will probably take more than one issue to complete..We would need other potentially useful modules such as analytics, customers etc.
The suggestion does open up one interesting avenue which may be to start off by dropping the standard ModelAdmin templates completely and designing from scratch, since a dashboard would probably necessitate that anyway (I dont think wagtail has any particular support for this use case - we should ask a wagtail developer to confirm).

It also reminds me that adding/editing shipping rates is also something that needs to be addressed; there is currently only the model.

@JamesRamm
Copy link
Collaborator Author

The changes being discussed here: wagtail-nest/wagtail-modeladmin#10 would probably be beneficial for this...

@alexfromvl
Copy link
Contributor

The basis for the development of the dashboard suggest getting the wagtailadmin base template, add to it the necessary function. I did the pr with a basic template modeladmin/orders for development.

@JamesRamm
Copy link
Collaborator Author

Hi
Thanks for the pull request.
Since the Order model is a standard django model (rather than inheriting from Page), I was planning on using the ModelAdmin wagtail app, which will register generic models. See the 'order_template' branch for work I have done here.
With ModelAdmin, we can use the existing views (e.g IndexView) and just provide a better Detail view which allows fulfillment/refund/any other actions.
In the order_template branch, I have supplied such a template and work necessary to register it with ModelAdmin - I just need to provide handlers for the buttons then will make a pull request.
The front end for this view is written using react, which does add more work to the initial setup of longclaw for development, but it will make long term development and dynamic pages easier (hopefully).

If I understand correctly, your approach is the start of a more thorough overhaul overriding the root admin page itself?

@JamesRamm
Copy link
Collaborator Author

PR wagtail/wagtail#15 should cover enough to close this issue. I think wagtail/wagtail#14 which (if I understand) is the start of addressing a more general dashboard, might be more suited to a separate issue?

@JamesRamm
Copy link
Collaborator Author

@alexfromvl, could you (if still interested) open a separate issue to address your dashboard idea and current work on your PR?

@alexfromvl
Copy link
Contributor

@JamesRamm Ok.

@JamesRamm JamesRamm mentioned this issue Mar 24, 2017
JamesRamm pushed a commit that referenced this issue Jul 7, 2023
* Test py 37,38,39 (#1) dj 22 wt 211,212,213

* Fixes the template include issue with the lock/unlock menu item

* Pin the upper bound of the supported Wagtail version to 2.13 and Django to 2.2

* Add some initial documentation for development setup
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

2 participants