Hi,
Is there plans to introduce D8 version of this module?
We (@InternetDevels team) would like to help you with development of this version.
Thanks, @ipo4ka704.

CommentFileSizeAuthor
#20 google_analytics_reports-drupal_8_port-2220179-20.patch146.7 KBPlazik
#17 google_analytics_reports-drupal_8_port-2220179-17.patch140.2 KBzyyz
#16 google_analytics_reports-drupal_8_port-2220179-16.patch105.14 KBPlazik
#16 interdiff-2220179-13-2220179-16.txt64.51 KBPlazik
#13 google_analytics_reports-drupal_8_port-2220179-13.patch111.91 KBzyyz
#10 google_analytics_reports-drupal_8_port-2220179-10.patch229.39 KBPlazik
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Plazik’s picture

Plazik CreditAttribution: Plazik commented
Title: Drupal 8.x » Drupal 8 port
Version: 7.x-1.3 » 7.x-3.x-dev

Hi.
Yes, I'm plannig to do D8 port but after 7.x-3.0 version. Anyway, patches are welcomed.

ipo4ka704’s picture

ipo4ka704 CreditAttribution: ipo4ka704 commented

Ok I will start. Later I will give a patch.

yannisc’s picture

yannisc CreditAttribution: yannisc at Netstudio commented

Hi!

Any news on this?

Plazik’s picture

Plazik CreditAttribution: Plazik as a volunteer commented
Component: Reports module » All modules

Hi!

I'm planning to start D8 port after #1111358: Create automated tests. Any help appreciated.

Paulset’s picture

Paulset CreditAttribution: Paulset commented

Any news?

george.karaivanov’s picture

george.karaivanov CreditAttribution: george.karaivanov at Peytz & Co commented

Hi Guys.

Any news? Do you have something done? or at least something that you started working on

Plazik’s picture

Plazik CreditAttribution: Plazik as a volunteer commented

I haven't started yet.

george.karaivanov’s picture

george.karaivanov CreditAttribution: george.karaivanov at Peytz & Co commented

Hi @plazik

I've started working on porting. I've successfully ported module google_analytics_reports_api
You can pull code from http://cgit.drupalcode.org/sandbox-george.karaivanov-2713395/?h=8.x-1.x

Plazik’s picture

Plazik CreditAttribution: Plazik as a volunteer commented

@george.karaivanov thank you, I will take a look.

Plazik’s picture

Plazik CreditAttribution: Plazik as a volunteer commented
Status: Active » Needs work
FileSize
google_analytics_reports-drupal_8_port-2220179-10.patch229.39 KB

@george.karaivanov I did some changes from your port:

  • Replace \Drupal::state() with config files.
  • Remove "curl" from GoogleAnalyticsReportsApiFeed.
  • Replace array() with [].
  • Small refactoring.

Marked as "Needs work" because main module still needs port to D8.

Plazik’s picture

Plazik CreditAttribution: Plazik as a volunteer commented
Version: 7.x-3.x-dev » 8.x-3.x-dev
zyyz’s picture

zyyz CreditAttribution: zyyz at Peytz & Co commented
FileSize
google_analytics_reports-drupal_8_port-2220179-13.patch111.91 KB

@Plazik
D8 port of Google Analytics Reports module.

zyyz’s picture

zyyz CreditAttribution: zyyz commented
Status: Needs work » Needs review
Plazik’s picture

Plazik CreditAttribution: Plazik as a volunteer commented

@zyosarian thank you for the patch. I will review it.

Plazik’s picture

Plazik CreditAttribution: Plazik as a volunteer commented
Status: Needs review » Needs work
FileSize
interdiff-2220179-13-2220179-16.txt64.51 KB
google_analytics_reports-drupal_8_port-2220179-16.patch105.14 KB
  • Do not include LICENSE.txt to the patch. It will be added by Drupal.org bot automaticaly.
  • I've added missing permission for Google Analytics Reports module.
  • Added Top Sources, Top Keywords attachments for Google Analytics Summary view.
  • Google Analytics Reports settings saved in its own config instead of using Google Analytics Reports API config.
  • Clear cache after importing GA fields on admin page.

TODO:

  • Import GA fields after module enabling.
  • Port Google Analytics Reports block view.
zyyz’s picture

zyyz CreditAttribution: zyyz at Peytz & Co commented
FileSize
google_analytics_reports-drupal_8_port-2220179-17.patch140.2 KB

Includes the remaining tasks..
1. Import GA fields after module enabling.
2. Port Google Analytics Reports block view.
3. Drush integration.

zyyz’s picture

zyyz CreditAttribution: zyyz at Peytz & Co commented
Status: Needs work » Needs review
Plazik’s picture

Plazik CreditAttribution: Plazik as a volunteer commented

@zyosarian, thank you for the patch.

Does page view block work for you? It works for me only if Format is set to Table.
I can commit it if this is Views bug and not GAR bug.

Plazik’s picture

Plazik CreditAttribution: Plazik as a volunteer commented
FileSize
google_analytics_reports-drupal_8_port-2220179-20.patch146.7 KB

Small changes in GA views.

  • Plazik committed 6854cc1 on 8.x-3.x authored by zyosarian 
    Issue #2220179 by Plazik, zyosarian: Google Analytics Reports Drupal 8...
Plazik’s picture

Plazik CreditAttribution: Plazik as a volunteer commented
Status: Needs review » Fixed

Commited.

Thanks everyone.

Status: Fixed » Closed (fixed)

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