Problem/Motivation

We are hardcoding what drupal.org understands as "active" and "maintained" as taxonomy terms UUIDs. This hardcoding should not happen in Project Browser's code, so we need to change how we get those values.

Steps to reproduce

See the code of the plugin.

Proposed resolution

This new endpoint https://www.drupal.org/drupalorg-api/project-browser-filters will provide the values, so we can read them and store them in the cache whenever needed.

Issue fork project_browser-3485386

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

fjgarlin created an issue. See original summary.

fjgarlin opened merge request !611

fjgarlin’s picture

fjgarlin commented
Status: Active » Needs work

Note that this will need to be merged first #3309273: Remove unused properties from the Project class, then it will be ready for review.

fjgarlin’s picture

fjgarlin commented
Related issues: +#3309273: Remove unused properties from the Project class
fjgarlin’s picture

fjgarlin commented
Assigned: fjgarlin » Unassigned
Status: Needs work » Needs review

Tests are passing (new fixture was added), Drupalpod tests seem to work as expected, and I removed any dependency on other issues, so this can be merged first if needed. I commented on the related issues with small changes that would be needed if this one is merged first.

Needs review.

fjgarlin’s picture

fjgarlin commented

Rebased, changed the code that needed to be changed and pipelines are happy.

chrisfromredfin made their first commit to this issue’s fork.

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented
Status: Needs review » Fixed

g2g

Status: Fixed » Closed (fixed)

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