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

Search terms not persisting in admin reports #3906

Closed
brchristian opened this Issue Oct 23, 2013 · 2 comments

Comments

Projects
None yet
2 participants
@brchristian
Copy link
Contributor

brchristian commented Oct 23, 2013

I've just started up brand new 1-3-stable and 2-0-stable apps. When I go to http://0.0.0.0:4000/admin/reports/sales_total and perform a search, the date range I search for does not appear to persist, meaning that I have to keep entering it! Can these be changed to populate / persist?

@radar

This comment has been minimized.

Copy link
Member

radar commented Oct 28, 2013

submits bi-annual patch to Spree's built-in reporting functions

@radar

This comment has been minimized.

Copy link
Member

radar commented Oct 28, 2013

Submitted a patch to 1-3-stable, 2-0-stable, 2-1-stable and master.

Thanks!

@radar radar closed this in 5ff8a92 Oct 29, 2013

radar added a commit that referenced this issue Oct 29, 2013

Persist search fields across requests for sales total report
Fixes #3906

Conflicts:
	core/app/views/spree/admin/shared/_report_criteria.html.erb

tvdeyen added a commit to tvdeyen/spree that referenced this issue Nov 7, 2013

tvdeyen added a commit to tvdeyen/spree that referenced this issue Nov 7, 2013

tvdeyen added a commit to tvdeyen/spree that referenced this issue Nov 7, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment