-
Notifications
You must be signed in to change notification settings - Fork 25
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
chore: version bump 0.7.5 #1112
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
…ints (#1108) Description --- noticed the actual error from BE wasn't logged in sentry re. paper_wallet details, then noticed the amount of duplications - removed `Sentry.captureException(`s in each `catch` because we are already using the `captureConsoleIntegration` - added ignore for all the `"Already getting ...."` warnings from BE which are being logged as errors clogging up sentry - made sure all `console.error`s have a short description and are passing the error through - fixed lints
Added proper value for custom mode in frontend logic Co-authored-by: Brian Pearce <[email protected]>
Description --- There was an issue with auto-update checker executed multiple times which led to executing auto-update multiple times in the same time See logs: ``` [Log] INITIAL CHECK (useUpdateStatus.ts, line 129) [Log] INITIAL CHECK (useUpdateStatus.ts, line 129) [Log] INITIAL CHECK (useUpdateStatus.ts, line 129) [Info] New Tari Universe version available – Object (useUpdateStatus.ts, line 94) [Info] New Tari Universe version available – Object (useUpdateStatus.ts, line 94) [Log] INITIAL CHECK FLAG SET TO FALSE (useUpdateStatus.ts, line 131, x2) [Info] New Tari Universe version available – Object (useUpdateStatus.ts, line 94) [Info] Proceed with auto-update (useUpdateStatus.ts, line 99) [Info] Updater event – null – "PENDING" (useUpdateStatus.ts, line 121) [Log] INITIAL CHECK FLAG SET TO FALSE (useUpdateStatus.ts, line 131) [Info] Updater event – null – "PENDING" (useUpdateStatus.ts, line 121) ``` Co-authored-by: Brian Pearce <[email protected]>
Description --- Icons from the figma file Motivation and Context --- Added the icons as they are in figma The mac .icns will have padding again, so they don't look so oversized in the taskbar against the other icons How Has This Been Tested? --- Manually What process can a PR reviewer use to test or verify this change? --- Start the app and view the icons in the taskbar <!-- Checklist --> <!-- 1. Is the title of your PR in the form that would make nice release notes? The title, excluding the conventional commit tag, will be included exactly as is in the CHANGELOG, so please think about it carefully. --> Breaking Changes --- x - [x] None - [ ] Requires data directory on base node to be deleted - [ ] Requires hard fork - [ ] Other - Please specify <!-- Does this include a breaking change? If so, include this line as a footer --> <!-- BREAKING CHANGE: Description what the user should do, e.g. delete a database, resync the chain --> --------- Co-authored-by: Brian Pearce <[email protected]>
## Closes #1074 ### [ Summary ] - Unify formatting utilities into one file - Expand them to cover more cases - Make 2 decimals as default - Display full number of balance on hover over wallet card - Sticked to using `Intl.NumberFormat` api for localized formatting --------- Co-authored-by: Brian Pearce <[email protected]> Co-authored-by: brianp <[email protected]>
[#1013](#1013) Save issue reference in the AppStateStore so reference number is not disappearing after reopening settings. [Screencast from 11-22-2024 10:41:00 AM.webm](https://github.com/user-attachments/assets/933af946-f0c7-44ad-a799-66e2903966fb)
Co-authored-by: Brian Pearce <[email protected]>
[#851](#851) When WebGL not supported, disable visual mode, display snackbar and send info to the sentry ![Screenshot from 2024-11-22 00-20-42](https://github.com/user-attachments/assets/6369d6ae-8cff-4570-9bb1-25f5b0a479ac) ![Screenshot from 2024-11-22 00-24-31](https://github.com/user-attachments/assets/086a1bb1-e65a-4080-bb7f-aff147fb40b3) ![Screenshot from 2024-11-22 00-22-33](https://github.com/user-attachments/assets/22769ace-65bf-4e03-bf24-9c4c9dd45000) Co-authored-by: Brian Pearce <[email protected]>
Description --- We've found with @Misieq01 what's causing the `app_config` overrides. It's because older versions have: `custom_max_gpu_usage: Option<u32>`(which is `null` as default). It was later changed to `custom_max_gpu_usage: Vec<GpuThreads>`(which is `[]` by default). It was treated as types inconsistency by serde which led to using and saving default app config
stringhandler
approved these changes
Nov 22, 2024
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.
Description
Onwards and upwards