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

feat(terminal):auto switch namespace and modify prompt #3961

Merged
merged 1 commit into from
Sep 20, 2023

Conversation

xudaotutou
Copy link
Contributor

@xudaotutou xudaotutou commented Sep 19, 2023

🤖 Generated by Copilot at 2063155

Summary

🎨🛠️➕

This pull request improves the frontend of the sealos application by making the code more readable and the terminal more user-friendly. It formats the billing page component and adds a feature to the terminal component to automatically switch to and show the user's namespace. It also introduces a new variable nsid to store the namespace ID.

The billing page got a makeover
With object spread syntax all over
The terminal too
Has something new
It shows the nsid and namespace of the user

Walkthrough

  • Add nsid variable to store user's namespace ID from session (link)
  • Modify terminal command to switch and display user's namespace and echo feedback message (link)

@sealos-ci-robot
Copy link
Member

sealos-ci-robot commented Sep 19, 2023

🤖 Generated by lychee action

Summary

Status Count
🔍 Total 961
✅ Successful 399
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 561
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Patch has no changes to coverable lines.

📢 Thoughts on this report? Let us know!.

@zzjin zzjin added this to the v5.0 milestone Sep 20, 2023
@zzjin zzjin merged commit 6d52319 into labring:main Sep 20, 2023
23 checks passed
bxy4543 pushed a commit to bxy4543/sealos that referenced this pull request Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants