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

Edit your name when you're logged in with Google #1

Closed
ILIYANGERMANOV opened this issue Nov 3, 2021 · 0 comments · Fixed by #30
Closed

Edit your name when you're logged in with Google #1

ILIYANGERMANOV opened this issue Nov 3, 2021 · 0 comments · Fixed by #30
Assignees
@ILIYANGERMANOV
Labels
improvement Existing functionality to be improved
Milestone
Go Open Source!

Comments

@ILIYANGERMANOV
Copy link
Collaborator

Allow users to edit their names which are automatically fetched from their Google account after Google Login.

USE-CASE: See something different than "Hi, Iliyan" on the home screen
@ILIYANGERMANOV ILIYANGERMANOV added the improvement Existing functionality to be improved label Nov 3, 2021
@ILIYANGERMANOV ILIYANGERMANOV self-assigned this Nov 3, 2021
@ILIYANGERMANOV ILIYANGERMANOV added this to the Go Open Source! milestone Nov 5, 2021
@ILIYANGERMANOV ILIYANGERMANOV changed the title Edit user's names Edit your name when you're logged in with Google Nov 5, 2021
@ILIYANGERMANOV ILIYANGERMANOV mentioned this issue Nov 5, 2021
Merged
ILIYANGERMANOV added a commit that referenced this issue Nov 5, 2021
@ILIYANGERMANOV
Merge pull request #30 from ILIYANGERMANOV/develop
f9432de 
Milestone #1 Go Open Source!
@ILIYANGERMANOV ILIYANGERMANOV mentioned this issue May 14, 2022
Merged
7 tasks
ILIYANGERMANOV added a commit that referenced this issue Sep 18, 2022
@ILIYANGERMANOV
Fix CI crashing #1: reduce RAM usage
52c2e0a
ILIYANGERMANOV added a commit that referenced this issue Aug 21, 2023
@ILIYANGERMANOV
Fix internal_release.yml (attempt #1)
3da8453
ILIYANGERMANOV added a commit that referenced this issue Aug 25, 2023
@ILIYANGERMANOV
Fix the Telegram bot message (attempt #1)
eac28de
ILIYANGERMANOV added a commit that referenced this issue Sep 2, 2023
@ILIYANGERMANOV
Hook test #1
b15a5e2
ILIYANGERMANOV added a commit that referenced this issue Sep 2, 2023
@ILIYANGERMANOV
Git Hooks + dev exp improvements (#2536)
d90bcbf 
* Delete some unused code

* Remove "Final Steps" from the PR template

* Prevent duplicate PR runs

* Add detektFormat `pre-push` hook

* Add instructions to setup `.git/hooks`

* Test the pre-push hook

* Hook test #1

* WIP: Fix pre-push hook and detektFormat.sh

* Generate a new baseline.yml

* Fix the `lint` workflow to always upload Lint and Detekt result

* Fix the detekt/baselin.yml AS auto-format breaks it
ILIYANGERMANOV added a commit that referenced this issue Sep 3, 2023
@ILIYANGERMANOV
Fix the dependabot.yml (attempt #1)
8b144a9
ILIYANGERMANOV added a commit that referenced this issue Sep 3, 2023
@ILIYANGERMANOV
Devexp (#2552)
9a3ca1c 
* Create a separate "Detekt workflow"

* WIP: Configure dependabot

* Migrate Hilt to Version catalog

* Migrate Room to version catalog

* WIP: Massive legacy code cleanup

* Delete legacy code

* Migrate deps to version catalog

* Cleanup

* Refactor `libs.versions.toml`

* Update the dependabot configuration

* Fix the `dependabot.yml` (attempt #1)

* Fix the `dependabot.yml` (attempt #2)

* Fix the `dependabot.yml` (attempt #3)

* Fix the `dependabot.yml` (attempt #4)

* Fix the `dependabot.yml` (attempt #5)

* Update Deket baseline

* Don't send APK to Telegram if it's `dependabot`

* Bump the dependabot.yml open PRs limit to 10

* Add Gradle caching to the "Detekt" workflow
ILIYANGERMANOV added a commit that referenced this issue Sep 9, 2023
@ILIYANGERMANOV
Danger PR (fix Markdown #1)
9e53d49
ILIYANGERMANOV added a commit that referenced this issue Sep 9, 2023
@ILIYANGERMANOV
Developer's experience (#2582)
9f73c9d 
* Improve the "detektFormat" task

* Fix the pre-push hook

* Test the pre-push hook

* Fix file formatting

* Fix messed up files

* Setup DangerJS

* Attempt Danger to comment with the Detekt report

* Try to comment APK file on the PR using Danger

* Update apk.yml

* Remove 5k node modules form .gitignore

* WIP: Setup DangerJS + upgrade wrapper

* Danger PR

* Danger PR (fix Markdown #1)

* Add danger comment for the "APK" workflow

* Add Danger Detekt

* Fix the Danger comment in Detekt

* Update Danger

* Fix Danger "Detekt" message

* WIP Danger

* DangerJS devexp v0.0.1?

* Rerun CI

* Improve Danger

* Improve Danger comments

* Improve Danger comments 2

* Improve Detekt messages

* Danger messages

* Update Detekt baseline

* Add "What's changed?" tip

* PR template improvement
ILIYANGERMANOV added a commit that referenced this issue Sep 12, 2023
@ILIYANGERMANOV
Modularization #1: by screen (#2611)
4c0938e 
* WIP: Modularization (`:settings`)

* WIP: Modularization (extract `:screen-exchange-rates`)

* WIP: Modularization

* WIP: Modularization

* WIP: Modularization

* WIP: Modularization

* WIP: Modularization

* WIP: Modularization

* WIP: Modularization

* WIP: Modularization

* WIP: Modularization

* WIP: Modularization

* Success! Modularization finished. The app builds

* Update Detekt baseline
nvllz added a commit to nvllz/ivy-wallet that referenced this issue Jan 21, 2024
@nvllz
detekt fix Ivy-Apps#1
e8cc65c
ILIYANGERMANOV pushed a commit that referenced this issue Jan 21, 2024
@nvllz
Stickyheader overlap fixes (#2884)
fc9d3bf 
* add a double-row BalanceRow style, bring back the 4-character asterisk style

* abbreviate month name in stickyrow menu

* detekt fix #1

* detekt fix v2
ILIYANGERMANOV added a commit that referenced this issue Jan 27, 2024
@ILIYANGERMANOV
Fix the Integration tests workflow #1
dc94639
ILIYANGERMANOV added a commit that referenced this issue Jan 27, 2024
@ILIYANGERMANOV
Integration tests (#2896)
d41c3f5 
* WIP: Setup integration tests (androidTest)

* WIP: Setup Room migration tests

* Working room DB migration configuration

* Add DB migrations tests

* Add migration tests for 123 to 125

* Add GitHub Action workflow that runs the integration tests

* WIP: Fix Android linkage problems & the new workflow

* Add missing dependency

* Make the workflow run only for :ivy-data

* Fix Detekt errors

* Fix the Integration tests workflow #1

* Rename steps
ILIYANGERMANOV added a commit that referenced this issue Apr 9, 2024
@ILIYANGERMANOV
Attempt to fix GitHub Actions #1
8bd1d64
ILIYANGERMANOV added a commit that referenced this issue Apr 9, 2024
@ILIYANGERMANOV
Data layer bugfixes (#3111)
f1225b9 
* WIP: Property-based testing utils

* WIP: Property-based Arb setup

* Add support for arb income

* Add args for transaction

* WIP: Fix the `TransactionMapper`

* WIP: Add property-based tests

* WIP: Fix the transaction repository

* Refactor

* WIP: Fix tests

* Fix the `TransactionMapper` test

* Improve the `TransactionMapper` test

* WIP: Improve the transaction mapper tests

* Add property-based tests for the transaction mapper

* Refactor the transaction mapper

* Refactor to `TransactionMapper`

* Fix Detekt errors

* WIP: TransactionRepositoryImpl test

* WIP: TransactionRepositoryImplTest

* Speed-up tests

* WIP: Fix build errors

* Fix build errors

* Delete unnecessary method

* Remove unnecessary method

* Refactor the TransactionRepository

* Add tests for the transactions repository

* Add tests for the save methods

* Cover the TransactionRepository with tests (one failing)

* Fix failing test

* Fix Detekt

* Increase tests heap size

* Add tests and fix bug in the transaction mapper

* WIP: Fix transaction mapper entity<->domain isomorphism

* Fix the TransactionMapperPropertyTest

* Fix Detekt

* Attempt to fix GitHub Actions #1

* Revert useless fix

* Fix flaky tests

* Fix Detekt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ILIYANGERMANOV ILIYANGERMANOV

Labels
improvement Existing functionality to be improved
Projects
None yet
Milestone
Go Open Source!
Development

Successfully merging a pull request may close this issue.

Milestone #1 Go Open Source! Ivy-Apps/ivy-wallet
1 participant
@ILIYANGERMANOV