-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RWR 472 group version3 #848
base: develop
Are you sure you want to change the base?
Conversation
lazysoundsystem
commented
Nov 18, 2024
- wip
- more content to relationship name changes
- update: move to group v3 and run database updates
Build outputComposer Validate `success`PHP Lint `success`Docker Build `success`Environment Setup `success`Site Install `success`PHP Code Sniffer `success`Software VersionsPHP 8.2.27 (cli) (built: Dec 19 2024 07:26:44) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.2.27, Copyright (c) Zend Technologies
with Zend OPcache v8.2.27, Copyright (c), by Zend Technologies
with Xdebug v3.4.0, Copyright (c) 2002-2024, by Derick Rethans
Composer version 2.8.4 2024-12-11 11:57:47
Drupal Logs
PHP Logs
Pusher: @lazysoundsystem, Action: `pull_request`, Workflow: `Run tests``; |
This wasn't so complicated in the end, but did require a lot of manual finding and replacing - https://www.drupal.org/docs/extending-drupal/contributed-modules/contributed-module-documentation/group/group-v2v3-guides/upgrading-from-v2-to-v3 - I think it's okay to include in this week's deployment, but I don't think there's any harm in waiting so we can test it better. |
Best to let them test as well, like creating operations, clusters, adding members, ... |
Okay, putting the 'blocked' label till we've tested further. |
@attiks thanks for having a look - I tried deploying this to the stage instance, but hitting familiar errors for each of the base_field_override config files: https://jenkins.aws.ahconu.org/view/Response/job/response-stage-deploy/90/console
|
Fixed by a second deploy |
This has been tested and is ready for merging. I'm still a bit nervous about the double deploy, but I've tried it a few times and it works consistently. |