1.0 to 2.0 upgrade path

Created on 5 January 2022, over 2 years ago
Updated 10 April 2023, about 1 year ago

Problem/Motivation

We need to test, verify, fix and document the path from going from 1.x -> 2.x here

Remaining tasks

- Install 1.x and add some usage
- Install 2.x and observe issues
- Document issues to be fixed
- Test

✨ Feature request
Status

Fixed

Version

2.0

Component

Code

Created by

πŸ‡¨πŸ‡¦Canada b_sharpe

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • First commit to issue fork.
  • @nikathone opened merge request.
  • Status changed to Needs review over 1 year ago
  • πŸ‡¨πŸ‡¦Canada nikathone Ontario

    This is an initial upgrade path. It doesn't cover views or page manager. Also people who might have implemented the approach to add an extras column for as recommended in v1 are not taken into account.

  • First commit to issue fork.
  • πŸ‡¨πŸ‡¦Canada nikathone Ontario

    My last commit adds `entity_id` in the legacy schema to ensure that it stays as string. In #3270630 it was update to be an integer due to performance reasons but I would like us to keep it as string in the legacy plugin for this initial update path then open a follow up issue if the people who were using V1 face the performance problem.

  • πŸ‡ΈπŸ‡³Senegal diaodiallo

    Thanks Nia for the great work. This patch works for me as excepted for my use case, working with the legacy plugin to use my data from version one.

    Howeveer I found two potential issue which worth to discuss:

    1. When I enable the generic usage data and the generic usage data counter in plus the legacy plugin I have this exception:

    if (!$database_schema->fieldExists($table, $counterField)) {
      throw new \Exception('Usage Data table ' . $table . ' schema should have a counter ' . $counterField . ' column.');
    }
    

    Line 77 in srcUsageDataDatabaseStorage.php
    I think we need to test more to see how to have them all enabled without issue.

    2. I noticed when I visit/view a page where I have a media (I have a video) the module is adding the event_type download for the entity_type_id media row, it should be just 'view'
    I think this may be an issue for the version 2 of he module.

    Thanks.

  • First commit to issue fork.
  • Status changed to Fixed about 1 year ago
  • πŸ‡ΊπŸ‡ΈUnited States andileco

    Thank you for working on this upgrade path! Committed.

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

Production build 0.69.0 2024