Needs work
Project:
Google Analytics Reports
Version:
8.x-3.x-dev
Component:
Reports module
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
2 Sep 2019 at 17:47 UTC
Updated:
24 Dec 2019 at 12:10 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
hussainwebHere's a patch that fixes the issue for me.
I also tried reverting changes in the issue #2850463: Fix Error whenever saving either of the provided google_analytics_reports view without any customization but that broke those views. So, I am just going to leave that there. But I do think that that patch could be better and more generic and the problem lies elsewhere. Well, one issue at a time and this patch fixes the issue and also adds some necessary schema for views filters.
Comment #3
rajab natshahThank you Hussain for reporting and the patch!
I have seen your issue after
#3097496: Add config schema for google analytics date views filter plugin id
In my opinion, we are of need for more config schemas, but we need to manage some kind of update process for old views.
More testing will follow
Comment #4
rajab natshahComment #5
rajab natshahI have noticed that all of us are having a number of issues, each one fixed by some change of config or config schema
Reviewing all issues for this module and extra testing before we release the stable version
We could work on a number of automated testing cases.
I did a number of them in
Total Control Admin Dashboard
#3069156: Add Automated Testing for the Total Control Admin Dashboard module
Our test use for the module is in Varbase Total Control Dashboard

In my opinion, most of Drupal 8 contrib modules should stable before Drupal 9
Comment #6
rajab natshah#3102925: Add an automated testing for the [Google Analytics Reports] and API module