Problem/Motivation

Temporary code needs to be removed from the DrupalDotOrgJsonApi.php file:


    // @todo Remove the next three lines when https://www.drupal.org/project/project_browser/issues/3309273 is merged.
    $filter_values = $this->filterValues();
    $active_values = $filter_values['active'] ?? [];
    $maintained_values = $filter_values['maintained'] ?? [];

Issue fork project_browser-3487835

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

chrisfromredfin created an issue. See original summary.

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

shalini_jha’s picture

shalini_jha commented
Assigned: Unassigned » shalini_jha

shalini_jha opened merge request !618

shalini_jha’s picture

shalini_jha commented
Assigned: shalini_jha » Unassigned
Status: Active » Needs review

Removed the legacy code from DrupalDotOrgJsonApi.php. and raised a MR for this .moving this NR. Kindly review.

ankitv18’s picture

ankitv18 commented
Status: Needs review » Reviewed & tested by the community

@shalini_jha Thanks for working on this one ~~ changes looks quite straightforward.
FYI, these kind of issues should be picked by newbies first. :)
For now I'm marking this one RTBC.

shalini_jha’s picture

shalini_jha commented

@ankitv18,thanks for your review. Since I recently started working with the Project Browser issues,that is why I chose to pick this issue first to better understand.

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented
Status: Reviewed & tested by the community » Fixed

thanks!

Status: Fixed » Closed (fixed)

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