Problem/Motivation

The front end still communicates to the backend via the "drupal-org-proxy" endpoints. This was set in the very early days, but now that we have multiple sources, with `drupalorg` being just one of them, that URL doesn't seem adequate for "core".

Proposed resolution

I suggest changing it to "project_browser_json" or "project_browser_data" or something similar.

Issue fork project_browser-3462307

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.

chrisfromredfin’s picture

chrisfromredfin
he/him/his
Primary language English
Location Portland, Maine
commented
Issue tags: +Novice, +core-mvp, +stable blocker
shalini_jha’s picture

shalini_jha commented
Assigned: Unassigned » shalini_jha

shalini_jha opened merge request !617

shalini_jha’s picture

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

I have updated the route URL for drupal-org-proxy to /project-browser/data, and have made the necessary updates across all relevant occurrences. I have also generated the build to reflect these changes and raised a MR for review. Could you please review the changes and let me know if I have missed anything that needs to be updated?

ankitv18’s picture

ankitv18 commented

Changes looks good and project-browser/data makes more sense ~~ marking this one RTBC.

But please let these kind of issues picked by someone new to the drupal community.

ankitv18’s picture

ankitv18 commented
Status: Needs review » Reviewed & tested by the community
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.