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

fix: Add CleanedBasePath to GithubSetupData (#2141) #2142

Conversation

santelos
Copy link
Contributor

@santelos santelos commented Mar 14, 2022

Fixes: #2141

Background

Template for GithubAppSetupTemplate refers to the property CleanedBasePath of GithubSetupData struct that was undefined.

Changes:

  • Property CleanedBasePath was added into GithubSetupData struct
  • Property CleanedBasePath is assigned in server/controllers/github_app_controller.go with g.AtlantisURL.path
    Where g is GithubAppController struct

Test

  • Tested locally with modified for local dev docker-compose run atlantis server.
    Was able to see http://localhost:4141/github-app/setup page

@santelos santelos requested a review from a team as a code owner March 14, 2022 14:38
@davidham
Copy link

+1 to merge this

@dgokcin
Copy link

dgokcin commented Apr 2, 2022

trying to set up atlantis for the first time, also having this issue. if this is the fix, +1 for the merge.

@davidham
Copy link

davidham commented Apr 3, 2022

trying to set up atlantis for the first time, also having this issue. if this is the fix, +1 for the merge.

@dgokcin FWIW, I built an image from this branch, and when I ran it I was able to do the setup process, many thanks to @santelos for the fix.

@dgokcin
Copy link

dgokcin commented Apr 3, 2022

@davidham I also worked arount the issue by running atlantis server manually and going to localhost:4141/github-app/setup to retrieve the github related values. Afterwards I replaced the localhosts with my atlantis server url and it worked.

@jamengual jamengual merged commit 4c0a449 into runatlantis:master May 13, 2022
@jamengual
Copy link
Contributor

LGTM +1

@jamengual
Copy link
Contributor

krrrr38 pushed a commit to krrrr38/atlantis that referenced this pull request Dec 16, 2022
@nitrocode nitrocode added this to the v0.19.4 milestone Mar 23, 2023
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.

[GithubAppController] Can't evaluate field CleanedBasePath in type templates.GithubSetupData
5 participants