- Merge request !147Draft: Issue #3327584: New contribution records system ā (Closed) created by fjgarlin
- šŖšøSpain fjgarlin
Update: functionality-wise, everything is there. We're starting to review/improve and see if there is anything missing.
The D7 changes can be seen in the MR for this issue. The whole plan (D7 + D9 new system) is in the google doc linked in the issue description.
- šŖšøSpain fjgarlin
fjgarlin ā changed the visibility of the branch 7.x-3.x to hidden.
- Merge request !273Resolve #3327584 "Contribution records new system" ā (Closed) created by fjgarlin
- Status changed to Needs work
10 months ago 9:46am 24 July 2024 - šŖšøSpain fjgarlin
The new MR contains the code from the old one as it was 2 years behind and rebase didn't work straightaway.
I will need to deploy this again to a D7 site and test.
- šŖšøSpain fjgarlin
Updated deployment information, queues to set up, etc.
- šŖšøSpain fjgarlin
fjgarlin ā changed the visibility of the branch 3327584-contribution-records-new-system to hidden.
- šŖšøSpain fjgarlin
With such a complex MR, it was easier to redo the whole thing than rebasing. New MR: https://git.drupalcode.org/project/drupalorg/-/merge_requests/332
- šŖšøSpain fjgarlin
I will try to split the MR into two:
- One for pushing data to D10
- One for reading that data from D10 - Merge request !336Resolve #3327584 "Receive contrib records from d10" ā (Open) created by fjgarlin
- šŖšøSpain fjgarlin
First part: Send data to new.drupal.org:
- MR: https://git.drupalcode.org/project/drupalorg/-/merge_requests/337- Settings on D7:
-- Setdrupalorg_token
to the same value as the one set in D10
-- Setup queuedrupalorg_issue_events
- Settings on D10:
-- Allow 'contribution_records' indrupalorg_allowed_content_types
-- Allow/contribution-record*
paths on Fastly
-- Setup environment variable so this line works:$config['drupalorg.settings']['token'] = getenv('DRUPALORG_TOKEN');
-- Setup queuecontribution_records_import_queue_worker
- šŖšøSpain fjgarlin
Added a configurable way to throttle the requests to the new system, in case we need it.
- šŖšøSpain fjgarlin
The second MR https://git.drupalcode.org/project/drupalorg/-/merge_requests/337 is mostly ready, but it relies on MR336 being merged first together with the deployment steps for that part.
After that, and when this is merged, we should be able to switch back and forth between the "legacy" and "modern" credit system with the following commands:
Use modern system:
drush vset drupalorg_credit_system 'modern' drush cc views
Switch back to legacy system:
drush vset drupalorg_credit_system 'legacy' drush cc views
A third part, and once the new system is stable and in use, will be to clean up the code from the legacy system and remove the switch, but we will go one step at a time.
- Status changed to Needs review
1 day ago 11:23am 15 May 2025 - šŖšøSpain fjgarlin
Updated the IS as we updated some of the token and headers names on both D7 and D10.