-
Notifications
You must be signed in to change notification settings - Fork 152
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
feat: added support for human readable predicates #994
feat: added support for human readable predicates #994
Conversation
Codecov Report
@@ Coverage Diff @@
## main #994 +/- ##
==========================================
+ Coverage 60.52% 60.73% +0.20%
==========================================
Files 178 178
Lines 5323 5379 +56
Branches 1511 1528 +17
==========================================
+ Hits 3222 3267 +45
- Misses 2076 2087 +11
Partials 25 25
|
Signed-off-by: wadeking98 <[email protected]>
7af4497
to
fd0f180
Compare
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.
Just a couple minor questions
@@ -137,6 +146,9 @@ const CredentialCard11: React.FC<CredentialCard11Props> = ({ | |||
resizeMode: 'contain', | |||
borderRadius: 10, | |||
}, | |||
attributeValueContainer: { | |||
width: '90%', |
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.
Just curious, what issue does this percentage width fix?
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.
it stops the text from overflowing the credential card, I tried a couple other approaches like flex wrap and whatnot but this is the best I could find
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.
I think the reason I had to resort to this approach is because the credential card has overflow: hidden
set for the watermark, so things like text will overflow and get cut off rather than wrapping
@@ -59,6 +60,7 @@ const SharedDataCard: React.FC<{ sharedData: GroupedSharedProofDataItem }> = ({ | |||
elevation: 5, | |||
}, | |||
cardAttributes: { | |||
width: '65%', |
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.
Same Q as above
@@ -391,6 +391,12 @@ const translation = { | |||
"CustomOfferParagraph2": "Não reconhece a organização. Verifique sua lista de Contatos. Você só recebe notificões de Contatos que você tenha adicionado.", | |||
}, | |||
"ProofRequest": { | |||
"PredicateGeDate": "is after (PB)", |
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.
"PredicateGeDate": "é posterior a",
"PredicateLeDate": "é anterior a",
"PredicateGe": "é maior ou igual a",
"PredicateGr": "é maior que",
"PredicateLe": "é menor ou igual a",
"PredicateLs": "é menor que",
Signed-off-by: wadeking98 <[email protected]>
…adeking98/aries-mobile-agent-react-native into feat-human-readable-predicates
…ent-react-native into feat-human-readable-predicates
Signed-off-by: wadeking98 <[email protected]>
Signed-off-by: wadeking98 <[email protected]>
824f099
to
72c1a70
Compare
Signed-off-by: wadeking98 <[email protected]>
…adeking98/aries-mobile-agent-react-native into feat-human-readable-predicates Signed-off-by: wadeking98 <[email protected]>
Signed-off-by: wadeking98 <[email protected]>
Summary of Changes
<=
->less than or equal to
,>
->greater than
, etc20020523
->May 23, 2002
Related Issues
N/A
Pull Request Checklist
Tick all boxes below to demonstrate that you have completed the respective task. If the item does not apply to your this PR check it anyway to make it apparent that there's nothing to do.
Signed-off-by
line (we use the DCO GitHub app to enforce this);If you have any questions to any of the points above, just submit and ask! This checklist is here to help you, not to deter you from contributing!
Pro Tip 🤓
PR template adapted from the Python attrs project.