-
Notifications
You must be signed in to change notification settings - Fork 3
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(net): github api #28
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clang-tidy made some suggestions
clang-tidy review says "All clean, LGTM! 👍" |
可以打开提交准备合入 |
clang-tidy review says "All clean, LGTM! 👍" |
Serein207
approved these changes
Aug 17, 2024
aurora0x27
pushed a commit
that referenced
this pull request
Aug 18, 2024
* feat:completed github api interface * style:Change code style to match clang-format --------- Co-authored-by: Serein <[email protected]>
zwow-cj
pushed a commit
that referenced
this pull request
Aug 19, 2024
* feat:completed github api interface * style:Change code style to match clang-format --------- Co-authored-by: Serein <[email protected]>
Serein207
added a commit
that referenced
this pull request
Sep 19, 2024
* feat(ui): complete main layout, add carousel. * feat(ui): add layout to "current activity" * feat(executor): different modes of `asyncExecute` using a timer One more parameter of `asyncExecute` when using a timer: TimerFlag flag TimerFlag::Once: perform the operation only once TimerFlag::Periodic: perform the operation periodically Implementation of `asyncExecuteByTimer` optimized `io_context` exposed for more flexibility * feat(executor): add execution strategy * feat: add view data to `ViewManager` (#17) * fix: added assert to `UiBridge::call` * refactor: added included header in `BasicView` for convenience * feat: added view data to `ViewManager` * feat(net): github api (#28) * feat:completed github api interface * style:Change code style to match clang-format --------- Co-authored-by: Serein <[email protected]> * docs(executor): optimize flag description * feat(net): evento api (#26) * feat:Preliminarily completed the implementation of the evento api interface * fix(net): Use `std::string` for all non-string arguments to meet the interface requirements of the URL library to resolve the compilation error. * fix(net)!: fix the false return type of refreshAccessToken and change the return type from void to bool for some function * fix(net): Fixed incorrect behavior in refreshAccessToken function. * refactor(net): Modified the return value settings of some functions to make the overall behavior more in line with server interface expectations * feat(net)!: Modified some function interfaces and implementations to align with the new backend interface specifications * feat(net): Implemented department-related functions. * fix(net): fix JSON parsing problem * feat(net)!: Changed the return value types of some functions to avoid passing arguments by reference. Additionally, renamed some incorrectly named variables in EventEntity to conform to backend interface specifications. * style(net): Modified code style to meet requirements. * feat(net): Add getEventList function * feat(net)!: Deprecated some functions and change the name of some variables and functions. * feat(net)!: Primarily enhanced the security of certain functions. Additionally changed the definition locations of some variables and optimized the parameter names of some functions. * feat(net): Initialize `std::optional<FeedbackEntity>` as `std::nullopt` in getUserFeedback * feat(ui): add scrollview to layout * feat/ui/discovery-page: add scarousel to scroollview * feat/ui/DiscoveryPage: accomplish dynamic grid using double for loop in VerticalLayout, but font not set yet * feat(ui): ui core multiple update (#31) * feat(ui): added state layer * feat(ui): modify overlay, improve menu and login * feat(ui): improve `MessageManager` * feat(ui): added developer shortcut to skip `LoginOverlay` Press Ctrl + Shift + Right click on shadow of `LoginOverlay` to skip login * feat: set default slint style to material * fix(ui): fix toast and login shadow (#33) * fix(ui): make login shadow opaque * fix(ui): make toast higher than overlay * feat: slint experimental feature * feat(ui): initialize sur-ui default theme * fix: crash showing message when another message on screen * feat(ui): optimize menu overlay * fix: open url failed on macos * build(deps): bump 3rdpart/sast-link-cxx-sdk from `9e2e743` to `2f62634` (#34) Bumps [3rdpart/sast-link-cxx-sdk](https://github.com/Serein207/sast-link-cxx-sdk) from `9e2e743` to `2f62634`. - [Release notes](https://github.com/Serein207/sast-link-cxx-sdk/releases) - [Commits](Serein207/sast-link-cxx-sdk@9e2e743...2f62634) --- updated-dependencies: - dependency-name: 3rdpart/sast-link-cxx-sdk dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * fix: make `openBrowser` inline, added dev shortcut Ctrl+Shift+Left click for skip login Ctrl+Shift+Right click for fake login (not trigger onLogin) * fix: menu expand animation lost * chore(cmake): remove useless variable * build(deps): bump 3rdpart/sast-link-cxx-sdk from `9e2e743` to `2f62634` (#35) Bumps [3rdpart/sast-link-cxx-sdk](https://github.com/Serein207/sast-link-cxx-sdk) from `9e2e743` to `2f62634`. - [Release notes](https://github.com/Serein207/sast-link-cxx-sdk/releases) - [Commits](Serein207/sast-link-cxx-sdk@9e2e743...2f62634) --- updated-dependencies: - dependency-name: 3rdpart/sast-link-cxx-sdk dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Serein <[email protected]> * feat(ui): finish setting page (#21) * feat:some basic laying * feat(ui): basic laying on the page * ui laying primary draft * feat(ui): callbacks and root * attempt to fill in .cc and .slint * callbacks fiied * feat: setting page * feat: change theme when app start * chore: clean up code * feat(ui): add scroll view in setting page --------- Co-authored-by: Serein <[email protected]> * feat(ui): `LoadingButton` component (#37) * feat: basic `LoadingButton` * feat: add custom animate * feat: export component in index, clean up code * refactor!: drop icon color auto switch function (#42) * refactor!: drop icon color auto switch * fix: correct existing `Image` reasons: - reduce resource file number and executable size when embed resource - color not exactly match environment (not the same color of text) - auto switch require duplicate files with little difference, and not fit into some special requirements perfectly - colorize support animation How to migrate: ``` Image { source: Token.image.icon.me; colorize: Token.color.on-surface; // should be added to all Image with source Token.image.icon.* } ``` Following icon keep previous behavior. - evento (bad support for `colorize`) BREAKING CHANGE: remove support for icon color auto-switch, please use `colorize` property of `Image` instead. * feat(ui): event card (#38) feat(ui): `Card`, `EventCard` , `EventCardGroup` and coversion function in `Convert.hh` * feat(ui): improve view data logic use std::array instead for convenience * feat(assets): remove useless images * feat(card): optimize animation * feat(card): optimize event card * feat(font): increase font size * feat(ui)!: add md3 button (#46) * feat(ui): added scrim, shadow color and elevation * feat(ui): improve menu shadow * feat(ui): improve state layer * feat(ui): improve `LoadingAnimation` * feat(ui): added md3 button BREAKING CHANGE: 目前 LoadingButton 已被删除,请使用 LoadingAnimation 和 Button 的组合 * feat(ui): complete main layout, add carousel. * feat(ui): add scrollview to layout * feat/ui/DiscoveryPage: accomplish dynamic grid using double for loop in VerticalLayout, but font not set yet * feat/ui/discovery-page: use component eventcardgroup * feat/ui/discovery-page: pull before push to remote * feat(ui): fix compile error and conflicts * feat/ui/discovery-page: modify component height * feat/ui/discovery-page: finish page display * feat/ui/discovery-page: seperate current and newest event * feat/ui/discovery-page: delete dead code * fix: correct about page link color * refactor: new login overlay * docs(readme): change to english * feat(cache): clear disk cache * fix(net)!: potential lifetime problem & modify api * perf(net): change buffer to local variable * fix(ui): load image logic * feat(ui): clean up code * fix * ci: limit thread number of build * build: replace platform macro * fix: remove redundant code due to merge * feat: controller logic * fix: miss header * perf: condition render info text --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Mairon <[email protected]> Co-authored-by: Serein <[email protected]> Co-authored-by: cEvolve05 <[email protected]> Co-authored-by: shuzhuxvchuang <[email protected]> Co-authored-by: Serein <[email protected]> Co-authored-by: ZhiweiCOfficial <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: DTPA101 <[email protected]> Co-authored-by: Mairon <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
API: