Skip to content
This repository has been archived by the owner on Sep 6, 2022. It is now read-only.

Details button now links to JSON URL #1077

Merged
merged 1 commit into from
Jul 14, 2018
Merged

Details button now links to JSON URL #1077

merged 1 commit into from
Jul 14, 2018

Conversation

KrauseFx
Copy link
Member

Fixes #1075

@@ -460,7 +460,7 @@ def save_build_status_locally!
def save_build_status_source!
status_context = project.project_name

build_path = FastlaneCI::BuildController.build_url(project_id: project.id, build_number: current_build.number)
build_path = FastlaneCI::BuildJSONController.build_url(project_id: project.id, build_number: current_build.number)
build_url = FastlaneCI.dot_keys.ci_base_url + build_path
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Side question, how does a User set up the ci_base_url?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FASTLANE_CI_BASE_URL - check out https://github.com/fastlane/ci/blob/master/docs/EnvironmentVariables.md for more info 😄

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems pretty important, should this be a separate global setting?

Copy link
Collaborator

@taquitos taquitos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for fixing this!
🎈:goat:

@KrauseFx KrauseFx merged commit 93814aa into master Jul 14, 2018
@KrauseFx KrauseFx deleted the build-json-url branch July 14, 2018 20:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants