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: link to test preview #1045

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

fix: link to test preview #1045

wants to merge 1 commit into from

Conversation

ofirc77
Copy link
Collaborator

@ofirc77 ofirc77 commented Feb 22, 2025

Description

Fix the broken link to test preview. The fix was made by changing some parts of the URL to consistent values in both workflows of the CI. We create 2 variables in one workflow and by uploading them to artifact we could access them in another workflow.

@ofirc77 ofirc77 self-assigned this Feb 22, 2025
@ofirc77 ofirc77 requested a review from NoamGaash as a code owner February 22, 2025 16:42
@NoamGaash NoamGaash marked this pull request as draft February 23, 2025 08:26
@NoamGaash NoamGaash removed their request for review February 23, 2025 08:26
@ofirc77 ofirc77 force-pushed the fix_s3_html_issue branch 6 times, most recently from 92c6904 to dafc916 Compare February 26, 2025 21:41
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.

1 participant