-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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(web-templates): use CleanedBasePath for static content #2079
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
When setting '--atlantis-url' this static content 404s. For example, we can see this when comparig: - /lock --> works seamless with a custom `--atlantis-url` - /jobs --> doesn't work when using a custom `--atlantis-url`
Thank you! |
jvrplmlmn
commented
Feb 24, 2022
@@ -102,7 +102,7 @@ func staticCssCustomCss() (*asset, error) { | |||
return nil, err | |||
} | |||
|
|||
info := bindataFileInfo{name: "static/css/custom.css", size: 6946, mode: os.FileMode(420), modTime: time.Unix(1641777138, 0)} | |||
info := bindataFileInfo{name: "static/css/custom.css", size: 6946, mode: os.FileMode(420), modTime: time.Unix(1635422826, 0)} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm unsure if this changes are required, I generated this file via:
make dist
chenrui333
approved these changes
Feb 27, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Thanks @jvrplmlmn! |
jvrplmlmn
added a commit
to jvrplmlmn/atlantis
that referenced
this pull request
Mar 1, 2022
A follow-up to runatlantis#2079 After upgrading to https://github.com/runatlantis/atlantis/releases/tag/v0.18.3 in our own infra, I noticed a minor issue with the links in the titles, they are still pointing to / even when '--atlantis-url' is provided.
chenrui333
pushed a commit
that referenced
this pull request
Mar 1, 2022
A follow-up to #2079 After upgrading to https://github.com/runatlantis/atlantis/releases/tag/v0.18.3 in our own infra, I noticed a minor issue with the links in the titles, they are still pointing to / even when '--atlantis-url' is provided.
krrrr38
pushed a commit
to krrrr38/atlantis
that referenced
this pull request
Dec 16, 2022
…is#2079) * fix(web-templates): use CleanedBasePath for static content When setting '--atlantis-url' this static content 404s. For example, we can see this when comparig: - /lock --> works seamless with a custom `--atlantis-url` - /jobs --> doesn't work when using a custom `--atlantis-url` * fix(web-templates): regenerate bindata_assetfs
krrrr38
pushed a commit
to krrrr38/atlantis
that referenced
this pull request
Dec 16, 2022
A follow-up to runatlantis#2079 After upgrading to https://github.com/runatlantis/atlantis/releases/tag/v0.18.3 in our own infra, I noticed a minor issue with the links in the titles, they are still pointing to / even when '--atlantis-url' is provided.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When setting '--atlantis-url' this static content 404s.
For example, we can see this when comparig:
--atlantis-url
--atlantis-url
Note that for the different templates, we are configuring
CleanedBasePath: s.AtlantisURL.Path
:atlantis/server/server.go
Lines 879 to 884 in 58e9b42
atlantis/server/controllers/locks_controller.go
Lines 82 to 94 in d914eaf
atlantis/server/controllers/jobs_controller.go
Lines 45 to 54 in 58e9b42
Credit goes to @iainlane 👍