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

Upload pier with SFTP #2062

Merged
merged 10 commits into from
Oct 2, 2023
Merged

Upload pier with SFTP #2062

merged 10 commits into from
Oct 2, 2023

Conversation

gdbroman
Copy link
Contributor

@gdbroman gdbroman commented Sep 28, 2023

Ticket: RE-487

Description

Here we implement SFTP in the BYOP onboarding flow in favor of the less reliable web upload.

Docs have been updated: https://docs.holium.com/main/hosting/upload-pier-with-sftp

Affected steps

Disclaimer Upload
CleanShot 2023-09-28 at 16 53 24@2x
redirect.mp4

Reviewer Checklist

  • Pipeline passes
  • Docs have been added / updated
  • Tests have been added / updated
  • Has been refactored if necessary

@gdbroman gdbroman self-assigned this Sep 28, 2023
@vercel
Copy link

vercel bot commented Sep 28, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Ignored Deployments
Name Status Preview Comments Updated (UTC)
holium-com ⬜️ Ignored (Inspect) Visit Preview Sep 30, 2023 3:42pm
hosting-holium-com ⬜️ Ignored (Inspect) Visit Preview Sep 30, 2023 3:42pm
join-holium-com ⬜️ Ignored (Inspect) Visit Preview Sep 30, 2023 3:42pm

@gdbroman gdbroman changed the title Upload pier with SFTP Replace web upload with SFTP Sep 28, 2023
@gdbroman gdbroman changed the title Replace web upload with SFTP Upload pier with SFTP Sep 28, 2023
@gdbroman
Copy link
Contributor Author

gdbroman commented Oct 2, 2023

Just ran the request flow by Johnathan and Liam. Merging, and doing two tests in prod: one monthly byop-p, and one monthly planet.

@gdbroman gdbroman merged commit fe3e7c1 into master Oct 2, 2023
3 checks passed
@gdbroman gdbroman deleted the RE-487-sftp branch October 2, 2023 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hosting anything related to new hosting system or site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant