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

Change the landing page banner. #4197

Merged
merged 1 commit into from
Sep 14, 2023
Merged

Change the landing page banner. #4197

merged 1 commit into from
Sep 14, 2023

Conversation

rochacbruno
Copy link
Member

@rochacbruno rochacbruno commented Sep 7, 2023

https://github.com/ansible/galaxy-deploy/pull/226 makes it possible to use /statc/images/ path from s3

@github-actions github-actions bot added the backport-4.7 This PR should be backported to stable-4.7 (2.4) label Sep 7, 2023

Verified

This commit was signed with the committer’s verified signature.
rochacbruno Bruno Rocha
No-Issue
src='https://www.ansible.com/hubfs/rh-2023-summit-ansiblefest-ansible-galaxy-site-200x200.png'
width='50%'
alt='Generative Ai, The Ansible way. Try Ansible Lightspeed with IBM watsonx Code Assistant'
src='/static/images/LightspeedGalaxyAd1.png'
Copy link
Collaborator

@himdel himdel Sep 10, 2023

Choose a reason for hiding this comment

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

The file won't make it to /static/ in the build unless it's imported somewhere..

Suggested change
src='/static/images/LightspeedGalaxyAd1.png'
src={LightspeedGalaxyAd1}

with an import LightspeedGalaxyAd1 from 'src/../static/images/LightspeedGalaxyAd1.png'; on the top.

EDIT: nope

Copy link
Collaborator

@himdel himdel Sep 10, 2023

Choose a reason for hiding this comment

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

... Unless you already have the files available there, in which case, LGTM 👍 :)

(sorry, confusion between /static/ on the server and static/ in the repo)

@himdel himdel added community comunity mode feature and removed backport-4.7 This PR should be backported to stable-4.7 (2.4) labels Sep 11, 2023
@himdel himdel merged commit ba390fe into ansible:master Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community comunity mode feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants