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

Comments

plazik’s picture

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 commented

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

yannisc’s picture

yannisc commented

Hi!

Any news on this?

plazik’s picture

plazik 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 commented

Any news?

george.karaivanov’s picture

george.karaivanov commented

Hi Guys.

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

plazik’s picture

plazik commented

I haven't started yet.

george.karaivanov’s picture

george.karaivanov 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 commented

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

plazik’s picture

plazik commented
Status: Active » Needs work
StatusFileSize
new 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 commented
Version: 7.x-3.x-dev » 8.x-3.x-dev
zyyz’s picture

zyyz commented
StatusFileSize
new google_analytics_reports-drupal_8_port-2220179-13.patch111.91 KB

@Plazik
D8 port of Google Analytics Reports module.

zyyz’s picture

zyyz commented
Status: Needs work » Needs review
plazik’s picture

plazik commented

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

plazik’s picture

plazik commented
Status: Needs review » Needs work
StatusFileSize
new interdiff-2220179-13-2220179-16.txt64.51 KB
new 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 commented
StatusFileSize
new 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 commented
Status: Needs work » Needs review
plazik’s picture

plazik 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 commented
StatusFileSize
new 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 commented
Status: Needs review » Fixed

Commited.

Thanks everyone.

Status: Fixed » Closed (fixed)

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