Trust protect feature #452
Conversation
Frankdevlopes
commented
Oct 18, 2025
Frankdevlopes
commented
- Added 'applyTrustProtect' controller function to apply the update
- Integrated notification sending when Trust Protect adjusts review ratings
- Updated review API to support Trust Protect endpoints
- Refined membership API calls to align with Trust Protect logic
- Enhanced seller review screen to display Trust Protect button and supposrt the logic
- Updated translations (messages/en.json) to include new Trust Protect label and messages
…d the trustprotect button
…Mappi balance deduction
Frankdevlopes
requested review from
DarinHajou,
Junman140,
adisa39,
fredrick-amoako,
joey364,
omotoshoayomikun,
rjen77,
soleil00,
swoocn and
vrajasekhar1
and removed request for
adisa39
| toast.error(t('SCREEN.REVIEWS.USER_NOT_LOGGED_IN')); | ||
| return; | ||
| } | ||
|
|
There was a problem hiding this comment.
Hi @Frankdevlopes -consider using the app custom showNotification context instead of toast
| try { | ||
| // Step 1: Deduct 100 Mappi | ||
| const response = await deductMappi(100); | ||
|
|
There was a problem hiding this comment.
Consider moving the mappi deduction to the BE. It’s best to make use of the existing mappiDeduction helper function for consistency. Thanks.
There was a problem hiding this comment.
Thanks for the feedback!
I had initially opened this PR before the mappi consumption code was merged into the main branch. That’s why the deduction logic is still on the frontend. I’ll update the PR to move the mappi deduction to the backend and use the existing mappiDeduction helper for consistency.
|
|
||
| toast.success( | ||
| 'Trust Protect was successful! Review face changed to Sad.', | ||
| ); |
There was a problem hiding this comment.
Use showNotificationMessage from context file for persistent UX. Thanks
There was a problem hiding this comment.
I’ll update it to use showNotificationMessage from the context for a more consistent UX. Appreciate the guidance!
| logger.info(`Found ${data.receivedReviews.length} reviews received by Pioneer: ${searchBarValue}`); | ||
| logger.info( | ||
| `Found ${data.receivedReviews.length} reviews received by Pioneer: ${searchBarValue}`, | ||
| ); |
There was a problem hiding this comment.
Consider reducing the logged data in your final changes.
|
|
||
| // Deduct Mappi for Trust Protect | ||
| export const deductMappi = async ( | ||
| amount: number, |
There was a problem hiding this comment.
This won’t be necessary once mappi deduction is moved to the BE
