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(costcenter):measure by app #3983

Merged
merged 2 commits into from
Sep 26, 2023
Merged

Conversation

xudaotutou
Copy link
Contributor

@xudaotutou xudaotutou commented Sep 25, 2023

πŸ€– Generated by Copilot at c9e5f88

Summary

πŸ†•πŸ› οΈπŸŒ

This pull request adds new features, fixes, and improvements to the costcenter frontend. It adds new components for filtering, searching, and displaying the billing records, such as AppMenu, NamespaceMenu, SearchBox, SwitchPage, TypeMenu, and BillingDetails. It also adds localization support for English and Chinese languages, and updates the UI layout and style of the billing and valuation pages. It fixes some typos, bugs, and unused code in the frontend and the API endpoints, such as the CurrencySymbol component name, the queryType value, and the dns module import. It also adds some new properties to the BillingItem type, such as appType and name.

Sing, O Muse, of the valiant deeds of the costcenter team
Who toiled with skill and wisdom to improve the frontend code
They added new components, like AppMenu and BillingDetails
And fixed the bugs and errors that plagued the billing mode

Walkthrough

  • Add AppMenu component to render a popover button for selecting app type (link)
  • Add BillingDetails component to render a button and a modal for showing billing details (link)
  • Add BillingDetailsTable component to render a table for showing billing details (link)
  • Add DetailsIcon component to render an icon for showing details (link)
  • Add NamespaceMenu component to render a popover button for selecting namespace (link)
  • Add SearchBox component to render an input field and a button for searching by order number (link)
  • Add SwitchPage component to render a text and buttons for switching pages (link)
  • Add TypeMenu component to render a popover button for selecting billing type (link)
  • Modify CommonBillingTable component to show app type, costs, and billing details button (link, link, link, link)
  • Modify TransferBillingTable component to reduce padding of table cells (link)
  • Modify InOutTabPanel function to use appType state and prop for filtering and displaying billing records (link, link, link, link)
  • Modify TransferTabPanel function to use appType state and prop for filtering and displaying billing records (link, link, link)
  • Modify Billing function to style tabs and tab panels according to design (link, link)
  • Modify Valuation function to remove border from table header and adjust height (link, link, link, link)
  • Modify BasicTableHeaders constant to replace "Type" with "APP Type" (link)
  • Modify BillingItem type to add appType and name properties (link)
  • Modify /api/billing/getNamespaceList endpoint to fix queryType value (link)
  • Modify /api/price/index endpoint to remove console.log statement (link)
  • Modify common.json files to add new key-value pairs for localization (link, link, link, link)
  • Fix typo in CurrencySymbol component name (link)
  • Remove unused import statements and variables from various files (link, link, link, link)

@pull-request-size
Copy link

Whoa! Easy there, Partner!

This PR is too big. Please break it up into smaller PRs.

@sealos-ci-robot
Copy link
Member

sealos-ci-robot commented Sep 25, 2023

πŸ€– Generated by lychee action

Summary

Status Count
πŸ” Total 961
βœ… Successful 395
⏳ Timeouts 0
πŸ”€ Redirected 0
πŸ‘» Excluded 561
❓ Unknown 0
🚫 Errors 4

Errors per input

Errors in README_zh.md

Errors in README.md

Full action output

Full Github Actions output

}) {
const { isOpen, onClose, onOpen } = useDisclosure();
const { t } = useTranslation();
console.log(optional);
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove

@codecov
Copy link

codecov bot commented Sep 26, 2023

Codecov Report

All modified lines are covered by tests βœ…

πŸ“’ Thoughts on this report? Let us know!.

@c121914yu c121914yu merged commit bf46d39 into labring:main Sep 26, 2023
21 of 23 checks passed
bxy4543 pushed a commit to bxy4543/sealos that referenced this pull request Oct 13, 2023
* feat(costcenter):measure by app

* Update TypeMenu.tsx

---------

Co-authored-by: Archer <545436317@qq.com>
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