-
Notifications
You must be signed in to change notification settings - Fork 7
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
CU-86dt1fxtx-1 - Implement Contextual Messages on WcSdk on both sides #116
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
{ | ||
"changes": [ | ||
{ | ||
"packageName": "@cityofzion/wallet-connect-sdk-core", | ||
"comment": "Implemented a Contextual Messages on WcSdk", | ||
"type": "patch" | ||
} | ||
], | ||
"packageName": "@cityofzion/wallet-connect-sdk-core" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
{ | ||
"changes": [ | ||
{ | ||
"packageName": "@cityofzion/wallet-connect-sdk-react", | ||
"comment": "Implemented a Contextual Messages on WcSdk", | ||
"type": "patch" | ||
} | ||
], | ||
"packageName": "@cityofzion/wallet-connect-sdk-react" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
{ | ||
"changes": [ | ||
{ | ||
"packageName": "@cityofzion/wallet-connect-sdk-svelte", | ||
"comment": "Implemented a Contextual Messages on WcSdk", | ||
"type": "patch" | ||
} | ||
], | ||
"packageName": "@cityofzion/wallet-connect-sdk-svelte" | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,2 @@ | ||
export const ACCOUNT_PASSWORD: string = "Password: Not '12345' - because we're not living in 1995!" | ||
export const DAPP_METHOD_CONTEXT_MESSAGE: string = "Accept this and your balance goes 'to the moon'!" |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -31,6 +31,8 @@ export default function RequestCard( | |
props.closeRequest() | ||
} | ||
|
||
const contextualMessage = String(request.params.contextualMessage).trim() | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ensure that |
||
|
||
const items = Array.isArray(request.params) | ||
? request.params | ||
: request.params.invocations | ||
|
@@ -59,6 +61,12 @@ export default function RequestCard( | |
Neo3 | ||
</Text> | ||
</Flex> | ||
<Text fontSize="0.875rem" color="#888888" fontWeight="bold" mt="0.875rem"> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ensure that the color |
||
Method Contextual Message | ||
</Text> | ||
<Text fontSize="0.875rem" mt="0.5rem" data-testid="request-card__contextual-message"> | ||
{contextualMessage} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The |
||
</Text> | ||
<Text fontSize="0.875rem" color="#888888" fontWeight="bold" mt="0.875rem"> | ||
Method | ||
</Text> | ||
|
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.
Ensure that the test ID
request-card__contextual-message
is consistently used across all relevant test files to avoid potential discrepancies in element targeting.