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.

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

Add more hidden resources strings #470

Merged
merged 2 commits into from
Aug 26, 2024
Merged

Conversation

sergiupuhalschi-rdx
Copy link
Contributor

No description provided.

Comment on lines 61 to 64
"assetDetails_hideCollection_button": "Hide Collection",
"assetDetails_hideCollection_title": "Hide Collection",
"assetDetails_hideCollection_message": "Hide **%s** NFT Collection in your Radix Wallet? You can always unhide it in your account settings."
"assetDetails_hide_button": "Hide"
Copy link
Contributor

Choose a reason for hiding this comment

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

The following three should apply for this view, and follow the pattern used for hideasset_:

	"assetDetails_hideCollection_title": "Hide Collection",
	"assetDetails_hideCollection_message": "Hide **%s** NFT Collection in your Radix Wallet? You can always unhide it in your account settings."
	"assetDetails_hideCollection_button": "Hide"

Then, the button added here should just be:

       "assetDetails_hideCollectionButton": "Hide Collection",

Comment on lines 70 to 71
"transactionReview_hiddenAsset": "This asset is hidden and will not be visible in your Account",
"transactionReview_hiddenAssetWithdrawing": "This asset is hidden and is not visible in your Account"
Copy link
Contributor

Choose a reason for hiding this comment

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

Given we haven't merged this code yet, maybe change to:

	"transactionReview_hiddenAsset_deposit": "This asset is hidden and will not be visible in your Account",
	"transactionReview_hiddenAsset_withdraw": "This asset is hidden and is not visible in your Account"

@sergiupuhalschi-rdx sergiupuhalschi-rdx merged commit 1049db3 into main Aug 26, 2024
@sergiupuhalschi-rdx sergiupuhalschi-rdx deleted the sp/hidden-assets branch August 26, 2024 11:34
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants