Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display the link to the proposal PR using github_pr_url #20

Merged
merged 1 commit into from
Dec 6, 2023

Conversation

rjwebb
Copy link
Contributor

@rjwebb rjwebb commented Dec 6, 2023

This PR modifies the GET /api/v3/conversations endpoint to return a new field github_pr_url which is the URL of the pull request for the conversation proposal, if there is one. This is the URL of the PR on GitHub, using the FIP repo owner, FIP repo name and pull request number values. This PR also modifies the frontend so that it uses this value when displaying the link to the PR.

@rjwebb rjwebb changed the title expose github_pr_url which returns a link to the PR Display the link to the proposal PR using github_pr_url Dec 6, 2023
@rjwebb rjwebb marked this pull request as ready for review December 6, 2023 16:21
@rjwebb rjwebb merged commit a32a545 into main Dec 6, 2023
1 check failed
@rjwebb rjwebb deleted the rjwebb/dashboard-does-not-link-to-pr branch December 6, 2023 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant