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

doc(1.8.0): update backing image restore UI #1020

Merged
merged 2 commits into from
Dec 11, 2024

Conversation

houhoucoop
Copy link
Contributor

@houhoucoop houhoucoop commented Dec 10, 2024

Which issue(s) this PR fixes:

Issue longhorn/longhorn#9490

What this PR does / why we need it:

  • Update the backing image restore UI screenshot, the secret and secretNamespace fields are displayed only for encrypted backing images.
  • Remove IMPORTANT note, the secret and secretNamespace fields are automatically included in the BackingImage CR, user don't have to input manually.

Special notes for your reviewer:

Frontend PR: longhorn/longhorn-ui#822

Additional documentation or context

N/A

Signed-off-by: Yi-Ya Chen <yiya.chen@suse.com>
@houhoucoop houhoucoop requested a review from a team as a code owner December 10, 2024 03:21
@houhoucoop houhoucoop requested review from ChanYiLin, a110605, jillian-maroket and derekbit and removed request for jhkrug and jillian-maroket December 10, 2024 03:22
@derekbit derekbit requested review from jhkrug and removed request for derekbit December 10, 2024 03:22
Copy link

netlify bot commented Dec 10, 2024

Deploy Preview for longhornio ready!

Name Link
🔨 Latest commit 09e99ee
🔍 Latest deploy log https://app.netlify.com/sites/longhornio/deploys/6758e16766248a000807be19
😎 Deploy Preview https://deploy-preview-1020--longhornio.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@derekbit derekbit merged commit e774fba into longhorn:master Dec 11, 2024
6 checks passed
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.

4 participants