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:template develop envs & links #4073

Merged
merged 2 commits into from
Oct 12, 2023
Merged

feat:template develop envs & links #4073

merged 2 commits into from
Oct 12, 2023

Conversation

zjy365
Copy link
Contributor

@zjy365 zjy365 commented Oct 11, 2023

🤖 Generated by Copilot at 4925f6e

Summary

🌐🖱️🚀

This pull request adds and updates several features and components for the template app, such as custom icons, share and deploy links, and localization. It also fixes a bug in the dbprovider app's edit page. The main files affected are Header.tsx, index.tsx, and common.json in the frontend/providers/template folder, and edit/index.tsx in the frontend/providers/dbprovider folder.

RightContext menu
Icons, share, deploy, fix bug
Winter of updates

Walkthrough

  • Add custom icons and text for app information, share, and deploy features of the template app (link, link, link, link, link, link, link, link, link, link, link, link, link, link)
  • Fix bug that caused submitSuccess function to run without confirmation in EditApp component (link)
  • Add RightContext component to render custom context menu for desktop app (link, link, link)
  • Pass platform environment variables to getTemplateDataSource function in develop page (link)

Signed-off-by: jingyang <3161362058@qq.com>
Signed-off-by: jingyang <3161362058@qq.com>
@sealos-ci-robot
Copy link
Member

🤖 Generated by lychee action

Summary

Status Count
🔍 Total 942
✅ Successful 369
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 572
❓ Unknown 0
🚫 Errors 0

Full action output

Full Github Actions output

@codecov
Copy link

codecov bot commented Oct 11, 2023

Codecov Report

All modified lines are covered by tests ✅

📢 Thoughts on this report? Let us know!.

@c121914yu c121914yu merged commit b02cf53 into labring:main Oct 12, 2023
29 of 30 checks passed
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

3 participants