Skip to content

doc: add known issue for the upgrade#427

Merged
Vicente-Cheng merged 1 commit intoharvester:mainfrom
Vicente-Cheng:harvester-4477-doc
Sep 12, 2023
Merged

doc: add known issue for the upgrade#427
Vicente-Cheng merged 1 commit intoharvester:mainfrom
Vicente-Cheng:harvester-4477-doc

Conversation

@Vicente-Cheng
Copy link
Contributor

- add the known issue
  (https://github.com/harvester/harvester/issues/4477)

@harvesterhci-io-github-bot
Copy link
Collaborator

harvesterhci-io-github-bot commented Sep 8, 2023

Deploy Preview for harvester-preview ready!

Name Link
🔨 Latest commit 388d36d
🔍 Latest deploy log https://app.netlify.com/sites/harvester-preview/deploys/6500460b61818b5fbb9fdb29
😎 Deploy Preview https://6500460b61818b5fbb9fdb29--harvester-preview.netlify.app

Copy link
Member

@starbops starbops left a comment

Choose a reason for hiding this comment

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

LGTM, just some nits. Thank you for updating the upgrade known issue list.

Copy link
Contributor

@bk201 bk201 left a comment

Choose a reason for hiding this comment

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

lgtm!

Copy link
Contributor

@guangbochen guangbochen left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

Copy link
Contributor

@LucasSaintarbor LucasSaintarbor left a comment

Choose a reason for hiding this comment

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

Small NITs. LGTM, thanks!

    - add the known issue
      (harvester/harvester#4477)

Signed-off-by: Vicente Cheng <vicente.cheng@suse.com>
Co-authored-by: Guangbo <guangbo.chen@suse.com>
Co-authored-by: Kiefer Chang <kiefer.chang@suse.com>
Co-authored-by: Zespre Chang <starbops@hey.com>
Co-authored-by: Lucas Saintarbor <lucas.saintarbor@suse.com>
@Vicente-Cheng
Copy link
Contributor Author

Small NITs. LGTM, thanks!

Updated! Thanks @LucasSaintarbor

@Vicente-Cheng Vicente-Cheng merged commit a626243 into harvester:main Sep 12, 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.

7 participants