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.

Sign up for GitHub

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

Jump to bottom

chore: merge tag desktop-1.6.5 into develop #4189

Merged
merged 218 commits into from
Aug 11, 2022
Merged

chore: merge tag desktop-1.6.5 into develop #4189

merged 218 commits into from
Aug 11, 2022

Conversation

begonaalvarezd
Copy link
Member

@begonaalvarezd begonaalvarezd commented Aug 9, 2022 •
edited
Loading

Summary

Please summarize your changes, describing what they are and why they were made.

Notes:

Had to move haveStakingResultsCached to packages/shared/lib/wallet.ts because it was giving me circular dependency when it was defined in packages/shared/lib/participation/participation.ts

Changelog

- chore: merge tag desktop-1.6.5 into develop

Relevant Issues

Please list any related issues using development keywords.

Closes #3951

Testing

Platforms

Please select any platforms where your changes have been tested.

  • Desktop
    • MacOS
    • Linux
    • Windows
  • Mobile
    • iOS
    • Android

Instructions

Please describe the specific instructions, configurations, and/or test cases necessary to test and verify that your changes work as intended.

...

Checklist

Please tick all of the following boxes that are relevant to your changes.

  • I have followed the contribution guidelines for this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or modified tests that prove my changes work as intended
  • I have verified that new and existing unit tests pass locally with my changes
  • I have verified that my latest changes pass CI workflows for testing and linting
  • I have made corresponding changes to the documentation

begonaalvarezd and others added 30 commits February 17, 2022 17:30
@begonaalvarezd
feat: make selected wallet the main view (#2179)
c8fdd21
@begonaalvarezd
chore: merge remote-tracking branch 'origin/develop' into feat/single…
43ba336 
…-account
@begonaalvarezd
chore: move selectedAccount to global state
40cdd6e
@begonaalvarezd
fix: wrong prop name
3ba87f7
@begonaalvarezd
feat: remove dead contexts
796dd56
@begonaalvarezd
chore: refactor selectedAccountId with selectedAccount
b7e363d
@begonaalvarezd
fix: undefined selectedAccount in account navigation
435b6f2
@begonaalvarezd
chore: replace locale with localize in components below Wallet.svelte
c7e8a01
@begonaalvarezd
feat: remove unnecessary code around selectedAccount
276bfec
@begonaalvarezd
chore: remove unnecessary files
d8743b4
@begonaalvarezd
fix: missing import on prev commit
5c6d922
@Tuditi
Merge pull request #2361 from iotaledger/feat/global-selected-account
565f481 
chore: Move `selectedAccount` to global state
@begonaalvarezd
chore: remove unnecessary code around wallet contexts
beb8d4a
@begonaalvarezd
fix: broken line chart
3894808
@begonaalvarezd
fix: dropdown jumping border css
0db6f3b
@Tuditi
Merge pull request #2402 from iotaledger/feat/cleanup
f2b0682 
feat: remove dead code
@begonaalvarezd
fix: ledger regenerate address
e3117cb
@Tuditi
Merge pull request #2413 from iotaledger/fix/ledger
b244a61 
fix: ledger regenerate address
@begonaalvarezd
feat: add global top nav (#2427)
049e101 
* chore: move viewableAccounts & liveAccounts to Dashboard.svelte to expose globally

* feat: add top navigation with switcher modal

* feat: improve css

* feat: add max height to account switcher

* fix: account switcher css

* feat: rename AccountNavigation component to TopNavigation

* feat: add missing top nav border

* fix: restore DeveloperProfileIndicator

* fix: css typo

* fix: remove commented code

* fix: remove unused locale

* fix: dashboard flex height

* fix: typing
@begonaalvarezd
feat: add tooltips to sidebar tabs (#2430)
4f6983c 
* chore: move viewableAccounts & liveAccounts to Dashboard.svelte to expose globally

* feat: add top navigation with switcher modal

* feat: improve css

* feat: add max height to account switcher

* fix: account switcher css

* feat: rename AccountNavigation component to TopNavigation

* feat: add missing top nav border

* fix: restore DeveloperProfileIndicator

* fix: css typo

* feat: add sidebar tab component with tooltips

* fix: remove commented code

* fix: remove unused locale

* fix: dashboard flex height

* fix: typing

* fix: typing
@jeeanribeiro
feat: move stronghold status and action to profile modal (#2507)
2fbf07c 
* feat: adds stronghold status and action to profile modal

* fix: polishes layout

* fix: adds isSoftwareProfile condition to stronghold status and actions
@begonaalvarezd
chore: merge remote-tracking branch 'origin/develop' into feat/single…
ca21572 
…-account
@begonaalvarezd
fix: wrong import after merge
969aae4
@jeeanribeiro @begonaalvarezd
feat: move ledger HW state & refresh to profile modal (#2515)
c5b6d70 
* feat: move ledger state to profile modal

* feat: adds ledger icon next to profile name

* fix: fixes from code review

* fix: missing store isLedgerProfile subscription

* feat: add all ledger states to profile modal

* chore: remove old dashboard security file

* chore: remove old locales

Co-authored-by: Begoña Álvarez de la Cruz <[email protected]>
@jeeanribeiro @begonaalvarezd
feat: move backup to profile modal (#2520)
e0ac014 
* feat: adds extra small button class

* feat: moves backup to profile modal

* fix: changes reactive variables lines

* feat: show backup in profile modal only when unsafe & improve locales

Co-authored-by: Begoña Alvarez <[email protected]>
@jeeanribeiro @begonaalvarezd
feat: move update to profile modal (#2521)
c362f77 
* feat: adds backup to profile modal

* feat: display updater only when there is an update available

* feat: polish css

Co-authored-by: Begoña Alvarez <[email protected]>
@begonaalvarezd
Feat: single wallet dashboard (#2514)
3e2a23f 
* feat: adapt wallet balance ui

* fix: select account wrong reactivity

* fix: cleanup imports

* feat: show send/receive views on click

* feat: update tokens icons

* feat: add my assets

* feat: cleanup AccountTile

* feat: remove comment

* fix: account actions modal remount

* fix: delete account success leads to undefined selected account view
@begonaalvarezd
Feat: fix wrong merge & polish CSS (#2529)
39be417 
* fix: wrong profile modal logic for backup and ledger

* feat: polish css & fix wrong locale
chore: resolve conflicts
a7f97cf
@jeeanribeiro
feat: add token graph to the wallet graph (#2565)
513238b
maxwellmattryan and others added 16 commits July 27, 2022 15:08
@maxwellmattryan
refactor: staking result JSON file storage (#4050)
b4146df 
* refactor: use local result files

* chore: fix gitignore

* fix: add delete files back

* test: fix these

* fix: adjust formatting
@cvarley100 @maxwellmattryan
fix: reward discovery and ledger wallet discovery (#4064)
f1d8c3f 
* refactor: use local result files

* chore: fix gitignore

* fix: add delete files back

* fix: staking reward display

* fix: account load and initial sync logic

Co-authored-by: Matthew Maxwell <[email protected]>
@maxwellmattryan
chore: fix gitignore
e7170e3
@maxwellmattryan
fix: tooltip
4f83ad2
@maxwellmattryan
enhancement: adjust stylings of tooltip
360a6cf
@maxwellmattryan
chore: update build CI
8ce616e
@maxwellmattryan
fix: adjust build CI MacOS version
9c73ea1
@maxwellmattryan
chore: use macos 11 for CI
01a7e98
@Thoralf-M
Update wallet.rs rev (#4077)
ae79be6
@maxwellmattryan
chore: fix tooltip bug
cecdd3e
@cvarley100
fix: gap limits and syncing logic (#4078)
e655826 
* fix: pass correct gap limit for bg syncing

* improvement: update positioning of syncing message

* fix: reset syncing queue on logout

* fix: account sync gap limit

* fix: manual sync gap limit
@maxwellmattryan
fix: null account syncing queue store
58233b2
@cvarley100
fix: syncing issues (#4092)
505a057 
* fix: account update on balance change

* fix: rerender assets on staking cache completion

* fix: reset staking cache store on logout
@cvarley100
chore: update version
29f7f1f
@cvarley100
Merge branch 'release/desktop-1.6.4' of https://github.com/iotaledger…
2e9b84a 
…/firefly into release/desktop-1.6.4
@begonaalvarezd
chore: merge tag 'desktop-1.6.5' into merge/desktop-1.6.5
c25a3aa
@begonaalvarezd begonaalvarezd added the type:chore House-keeping etc. label Aug 9, 2022
@begonaalvarezd begonaalvarezd self-assigned this Aug 9, 2022
@begonaalvarezd begonaalvarezd marked this pull request as ready for review August 9, 2022 17:37
amadeu2
amadeu2 approved these changes Aug 10, 2022
View reviewed changes
Copy link
Contributor

@amadeu2 amadeu2 left a comment

Choose a reason for hiding this comment

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

Tested everything on Android and iOS from scratch and it's working perfect! 💯 Tested also Stronghold import with previous history and funds, it was recovered all as espected 💫😁

@begonaalvarezd begonaalvarezd merged commit dd7a904 into develop Aug 11, 2022
@begonaalvarezd begonaalvarezd deleted the merge/desktop-1.6.5 branch August 11, 2022 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amadeu2 amadeu2 amadeu2 approved these changes

@cvarley100 cvarley100 Awaiting requested review from cvarley100

@maxwellmattryan maxwellmattryan Awaiting requested review from maxwellmattryan

Assignees

@begonaalvarezd begonaalvarezd

Labels
type:chore House-keeping etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Task]: Balance is not updated after the stronghold import
9 participants
@begonaalvarezd @amadeu2 @Tuditi @jeeanribeiro @cvarley100 @maxwellmattryan @nicole-obrien @rajivshah3 @Thoralf-M