-
Notifications
You must be signed in to change notification settings - Fork 21
Feat/optional sender - squashed #520
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
Open
p2arthur
wants to merge
26
commits into
main
Choose a base branch
from
feat/optional-sender-improved
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 20 commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
9f35acb
chore(merge-main): merge latest from main with tailwind v4 to optiona…
p2arthur 0e78c0f
feat(optional_sender): Add optional sender at latest squashed to solv…
p2arthur 78166ad
chore(package-lock): update package lock to reflect dependencies from…
p2arthur 6c2ad60
feat(search_params) add optional sender with auto populate to search …
p2arthur 406f776
test(optional_sender): add optional sender to url params test
p2arthur a22d374
chore: add DataProvider to txn wizard test
PatrickDinh bf2ae35
chore: lint-fix
PatrickDinh fac30c1
feat(transaction-wizard): auto-populate sender from localnet dispense…
p2arthur 669d16f
fix(search_params): fix two tests that were failing only on CI/CD
p2arthur 799967c
fix(test): add racing condition to wait asset-opt out render before t…
p2arthur 02b70e6
fix(test): add racing condition to wait render before testing
p2arthur a874ce9
fix(test): add racing condition to wait render before testing
p2arthur d65e4b4
reafactor(non-null): remove non-null assertion from the transaction m…
p2arthur d7215ae
test(optional_sender): updated tests to use kmd instead of assuming t…
p2arthur cfe5917
chore(optional_sender): cleanup to remove repeated tests and optional…
p2arthur a7a8d85
fix(conflicts) fix merge conflicts from feature branch
p2arthur cc92c31
fix(conflicts) fix merge conflicts from feature branch
p2arthur bb29210
refactor(resolve_address): refactor the resolve sender address to sim…
p2arthur fb2a694
refactor(use-search-params): refactor use search params transaction t…
p2arthur 444a3cf
refactor(use-search-params): refactor transactions wizard page test t…
p2arthur 2c44d4d
chore: refactor the search params transactions hook
PatrickDinh 907ed2c
Merge pull request #531 from algorandfoundation/jotai-refactor
p2arthur aa66da7
refactor: rename resolveSenderAddress to resolveTransactionSender and…
p2arthur 4f57874
fix: fix failing tests for auto populating and simulating transactions
p2arthur 55986c0
fix: lint error blocking CI
p2arthur 8355622
test: add wait function to wait for the sender to get populated
p2arthur File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
21 changes: 21 additions & 0 deletions
21
src/features/accounts/components/transaction-sender-link.tsx
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,21 @@ | ||
| import { AddressOrNfdLink, AddressOrNfdLinkProps } from './address-or-nfd-link' | ||
| import { cn } from '@/features/common/utils' | ||
|
|
||
| export type Props = AddressOrNfdLinkProps & { autoPopulated?: boolean } | ||
|
|
||
| export default function TransactionSenderLink(props: Props) { | ||
| const { autoPopulated, className, ...rest } = props | ||
|
|
||
| return ( | ||
| <div className="flex items-center"> | ||
| <AddressOrNfdLink className={cn(className, autoPopulated && 'text-yellow-500')} {...rest} /> | ||
|
|
||
| {autoPopulated && ( | ||
| <span className="group ml-1 cursor-help text-yellow-500"> | ||
| <span>?</span> | ||
| <div className="absolute z-10 hidden rounded-sm border-2 border-gray-300/20 p-1 group-hover:block">auto populated</div> | ||
| </span> | ||
| )} | ||
| </div> | ||
| ) | ||
| } |
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
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.
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.
why was this added?