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

fix unnecessary query prefix for engine #102

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

Conversation

okankocyigit
Copy link

@okankocyigit okankocyigit commented Mar 20, 2017

this fix #97,

When we set the QueryStringPrefix as mentioned here, export not working anymore because export engine doesn't handle export action, querystring for engine is unmatching with engine type.

@JRyanJones
Copy link

I still get the same problem mentioned in #97 (3 years later). The export link takes you to a plain text page version of the "visible" rows of the data grid, opposed to downloading a CSV of the requested filtered rows. This only happens when using QueryStringPrefix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

multiple grid on same page and export csv
2 participants