Feat/ime 87 #62
Merged
Feat/ime 87 #62
Conversation
This file contains 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
Contributor
Ujjwal-Izyane
commented
Dec 18, 2024
Ujjwal-Izyane
commented
- Update UI to handle conversions
- Update graphql queries
- Update the graphs ui components
* feat: Enhance GraphQL queries with additional fields for transfer details * Added optional chaining to safely access the uid=1000(harlem) gid=1000(harlem) groups=1000(harlem),4(adm),20(dialout),24(cdrom),25(floppy),27(sudo),29(audio),30(dip),44(video),46(plugdev),116(netdev),1001(docker) property of in the TransferDetails component. This prevents runtime errors when is null or undefined. * Added optional chaining to safely access the uid=1000(harlem) gid=1000(harlem) groups=1000(harlem),4(adm),20(dialout),24(cdrom),25(floppy),27(sudo),29(audio),30(dip),44(video),46(plugdev),116(netdev),1001(docker) property of in the TransferDetails component. This prevents runtime errors when is null or undefined. * merged the code in preparation for enhancements * Make fx fields and buttons hidden * task/ime-242-verify_ui * updated technical details fields * task/ime-242-verify_ui * updated technical details mapping * updated mapping * updated mapping * Fix formatting issue * task/ime-242-verify_ui * task/ime-242-verify_ui * updated the models and removed Transaction model * feat: Update GraphQL queries to align with mock data structure * feat: Update GraphQL queries to align with mock data structure * changes * feat: Update GraphQL queries to align with mock data structure * updated types * updated mapping to types * updated mocks * updated dashboard ui * update: transfer summary dashboard ui and mapping * UI mapping modal * mapping done for modals * fixed lint errors * updated ui * update: transfer summary dashboard UI - Create new GraphQL query - Map UI to the updated GraphQL schema - Refine logic to display successful and failed transactions in doughnut charts - Adjust dashboard UI alignment * completed mapping * updated payer information modal * fix: white screen bug due to mock.ts * update: transfer query to also add events fields * fix: payer and payeeDFSP undefined bug in tranfer tables * updated final ui * fix: payerModal showing wrong fspId fix: settlement batch id null * update: UI mapping to new graphql changes updated graphl query types * update: transfer query to fetch FxQuote and FxTransfer events * add: settlement batch id - updated transfer query - updated graphql types - updated ui mapping to new graphQl * fix: conversion terms source currency maaping * fix: lint errors --------- Co-authored-by: Mayhem <[email protected]> Co-authored-by: Naph <[email protected]> Co-authored-by: Akhilesh Rawat <[email protected]>
* feat: Enhance GraphQL queries with additional fields for transfer details * Added optional chaining to safely access the uid=1000(harlem) gid=1000(harlem) groups=1000(harlem),4(adm),20(dialout),24(cdrom),25(floppy),27(sudo),29(audio),30(dip),44(video),46(plugdev),116(netdev),1001(docker) property of in the TransferDetails component. This prevents runtime errors when is null or undefined. * Added optional chaining to safely access the uid=1000(harlem) gid=1000(harlem) groups=1000(harlem),4(adm),20(dialout),24(cdrom),25(floppy),27(sudo),29(audio),30(dip),44(video),46(plugdev),116(netdev),1001(docker) property of in the TransferDetails component. This prevents runtime errors when is null or undefined. * merged the code in preparation for enhancements * Make fx fields and buttons hidden * task/ime-242-verify_ui * updated technical details fields * task/ime-242-verify_ui * updated technical details mapping * updated mapping * updated mapping * Fix formatting issue * task/ime-242-verify_ui * task/ime-242-verify_ui * updated the models and removed Transaction model * feat: Update GraphQL queries to align with mock data structure * feat: Update GraphQL queries to align with mock data structure * changes * feat: Update GraphQL queries to align with mock data structure * updated types * updated mapping to types * updated mocks * updated dashboard ui * update: transfer summary dashboard ui and mapping * UI mapping modal * mapping done for modals * fixed lint errors * updated ui * update: transfer summary dashboard UI - Create new GraphQL query - Map UI to the updated GraphQL schema - Refine logic to display successful and failed transactions in doughnut charts - Adjust dashboard UI alignment * completed mapping * updated payer information modal * fix: white screen bug due to mock.ts * update: transfer query to also add events fields * fix: payer and payeeDFSP undefined bug in tranfer tables * updated final ui * fix: payerModal showing wrong fspId fix: settlement batch id null * update: UI mapping to new graphql changes updated graphl query types * update: transfer query to fetch FxQuote and FxTransfer events * add: settlement batch id - updated transfer query - updated graphql types - updated ui mapping to new graphQl * fix: conversion terms source currency maaping * fix: lint errors * fix: yarn build bug - updated mock.ts object types --------- Co-authored-by: Mayhem <[email protected]> Co-authored-by: Naph <[email protected]> Co-authored-by: Bupe <[email protected]>
vijayg10
reviewed
Dec 20, 2024
CHANGELOG.md
Outdated
| @@ -1,123 +1,107 @@ | |||
| # Changelog: [mojaloop/reporting-hub-bop-trx-ui](https://github.com/mojaloop/reporting-hub-bop-trx-ui) | |||
|
|
|||
There was a problem hiding this comment.
This changelog is automatically generated one. We don't need to lint check it. I think you are using some linting tool and we should configure it to check only specific folder like src.
There was a problem hiding this comment.
Fixed this one , manually removed this line for now
LICENSE.md
Outdated
Comment on lines
5
to
6
| The Mojaloop files are made available by the Mojaloop Foundation under the Apache License, Version 2.0 (the "License") and you may not use these files except in compliance with the License. | ||
| The Mojaloop files are made available by the Mojaloop Foundation under the Apache License, Version 2.0 | ||
| (the "License") and you may not use these files except in compliance with the [License](http://www.apache.org/licenses/LICENSE-2.0). |
There was a problem hiding this comment.
Please do not change the license file
There was a problem hiding this comment.
Fixed this , copied the original license of the master here
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.