1️⃣ I have not found the time to advance the Drupal 9 API endpoint site, and the next specific thing is writing a migration to migrate releases into it (it's a node type). Does anyone with any migration experience want to chip in here? You would need to sign the NDA with the Drupal Association, but then we can get you what you need.
| Björn Brala (bbrala) | That shouldn't be too hard right? Does the checkmark @Rajab Natshah mean you are gonna help? |
| Rajab Natshah | @Björn Brala (bbrala) I signed the NDA with the Drupal Association.Yes, I like to help.To me, it feels that this is a big task. We should sync more with the team who worked on building the new Drupal.org with Drupal 9.Older members or more experts on the current Drupal.org are at the front. |
| Björn Brala (bbrala) | Ok cool, just checking for clarification :+1: |
3️⃣ Can someone spin up a local to test and see if they can replicate @gaurav.kapoor’s issue in #6 here? #3252674: No pre-checks on compatible releases which results on Notice: Undefined index: date in DrupalOrgClient
| chrisfromredfin | Could potentially just do this in a GitPod using the main button on the project page. |
| chrisfromredfin | Yes I can replicate this. I'll start digging. |
| chrisfromredfin | This is because "The website encountered an unexpected error. Please try again later." is being returned and it's expecting JSON but it starts with T (The). This is because of Error: Class 'Kevinrob\GuzzleCache\CacheMiddleware' not found in Drupal\project_browser\DrupalOrg\DrupalOrgClient->createGuzzleClient() (line 58 of /var/www/html/web/modules/contrib/project_browser/src/DrupalOrg/DrupalOrgClient.php) |
| gaurav.kapoor | This issue has been reported in some other PHP projects as well. https://github.com/launchdarkly/php-server-sdk/pull/78 |
| gaurav.kapoor | Maybe some dependency dropped it from composer.json without handling removal or usage. (edited) |
| chrisfromredfin | our composer.json does explicitly require it, but it's not ending up in /verndor when I run composer install... ? |
| chrisfromredfin | oh. because i just cloned the module into the site. I should add it with composer. |
| chrisfromredfin | yeah so it's actually working. |
| chrisfromredfin | With that said, generally any kind of error / 500 back from the Drupal side API endpoint is going to break in that way. We should probably add some smartness to that to check the return code maybe or something. |
| gaurav.kapoor | I'll check it again as well, maybe my local had some other issue. |
| chrisfromredfin | if you can find where that is and you can file an issue, that's great. |
| chrisfromredfin | also a lot of the other issues I was having where it was requesting plain /project without parameters and taking forever were rectified with Najab's patch from 2️⃣ I'll reivew and merge that one I think/hope, except I have to get a DDEV meeting. :stuck_out_tongue: |
5️⃣ Project summary and the full project body description ( Noticed the fixtures for content, almost all projects with HTML in the body )https://git.drupalcode.org/project/project_browser/-/blob/1.0.x/fixtures... to have a field_project_browser_summary ( can be changed for sure) and do this fixture in the current Drupal.org system.Create the field text-only summary fieldLet maintainers set a value for it in their projects.A cron job to copy the current summary or body and trim it and clear the HTML from it to start with.Remove the fixture file and code from the Project Browser module(edited)
| chrisfromredfin | I don't disagree that this is a huge improvement; SBS has it in their backlog to come up with a suggested structure on d.o for new fields we can/should use to better-present that page/data. Maybe file an issue, or add this commentary if there's an existing issue about ideas for re-organizing the drupal.org "project" content type. cc @leslieg |
| Rajab Natshah | Noted; @chrisfromredfin I will follow on that |
Participants:
Björn Brala (bbrala), Rajab Natshah, chrisfromredfin, gaurav.kapoor
Comments
Comment #2
NadiaFaucon commentedStatus change from "Active" to "NeedsReview"
Comment #7
leslieg commentedComment #8
leslieg commented