Hi,

Thanks for the module.

I'm am trying to get it to work on 8.5.1 and it displays, Client error.

The logs show:

Symfony\Component\HttpKernel\Exception\NotAcceptableHttpException: No route found for the specified format html. in Drupal\Core\Routing\RequestFormatRouteFilter->filter() (line 55 of /mnt/www/html/metrotas4dev/docroot/core/lib/Drupal/Core/Routing/RequestFormatRouteFilter.php).

Any ideas how to fix?

Best, JIm

CommentFileSizeAuthor
#15 2957100-15.patch444 byteshardikpandya
#13 2957100-11.patch432 byteshardikpandya
#11 interdiff_8-11.txt475 byteshardikpandya
#8 client-error-2957100-8.patch430 bytesgovind.maloo

Comments

jwilde created an issue. See original summary.

juliakoelsch’s picture

juliakoelsch commented

We have a view set up to export a csv using this module, and it was working correctly prior to Wednesday when we upgraded to 8.5.1. Now we are now experiencing the same error.

devoleksiy’s picture

devoleksiy commented

I also have such a mistake. drupal 8.5.1
module fb_instant_articles does not work

kducharm’s picture

kducharm commented

This is appearing due to changes in 8.5.1 - see https://www.drupal.org/project/views_data_export/issues/2957085

sassafrass’s picture

sassafrass commented

I am also experiencing this issue after an update to Drupal 8.5.1

rubix05’s picture

rubix05 commented

I also am experiencing this issue. I was able to get around it by adding
?_format=csv to the end of the url path.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Priority: Critical » Normal
Issue tags: +Novice

Lowering priority. Seems that by adding _format=csv, the error goes away. The next actionable way to actually close this would be to add something to the readme about doing that. Adding a line or two to the readme seems easy enough, so tagging novice.

govind.maloo’s picture

govind.maloo commented
Status: Active » Needs review
StatusFileSize
new client-error-2957100-8.patch430 bytes
heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs review » Needs work 
+++ b/README.md
@@ -53,7 +53,7 @@ Configuration
+Visit the path that you set for the view with additonal query string "\_format=csv"

I think ?_format=csv is more correct.

hardikpandya’s picture

hardikpandya commented
Assigned: Unassigned » hardikpandya
hardikpandya’s picture

hardikpandya commented
Assigned: hardikpandya » Unassigned
Status: Needs work » Needs review
StatusFileSize
new interdiff_8-11.txt475 bytes

Adding the interdiff file.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs review » Needs work

That isn't an interdiff, it's a patch. But with a txt extension. Can you provide in the right format so the test bot will pick it up, please :)

hardikpandya’s picture

hardikpandya commented
Status: Needs work » Needs review
StatusFileSize
new 2957100-11.patch432 bytes

Yes, here it is!

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs review » Needs work 
+++ b/README.md
@@ -53,7 +53,7 @@ Configuration
+Visit the path that you set for the view with additonal query string "?_format=csv"

Nit: 80 characters :)

Nit: It would sound like smoother English with some more glue words in there. Maybe something like: "Visit the path that you set for the view and add this additional query string: "?_format=csv"."

hardikpandya’s picture

hardikpandya commented
Status: Needs work » Needs review
StatusFileSize
new 2957100-15.patch444 bytes
heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs review » Reviewed & tested by the community

Good docs improvements. Looks good now.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.