Skip to content

get users details before data review notification - #587

Merged
Ramon-Jimenez merged 2 commits into
developmentfrom
fix/datareviewer-error-869eb6tj8
Aug 5, 2026
Merged

Ramon-Jimenez merged 2 commits into
developmentfrom
fix/datareviewer-error-869eb6tj8

Conversation

@eperedo

@eperedo eperedo commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

📌 References

📝 Implementation

User details is not included in /api/metadata/userRoles:fields=users[userCredentials]. To fix this we're getting the users details from the /api/users endpoint.

🔥 Notes for the reviewer

Data-Management-App.mp4

🎨 Screenshots

@eperedo
eperedo requested a review from Ramon-Jimenez July 30, 2026 19:11
Comment thread src/models/__tests__/ProjectNotification.spec.ts Fixed
@bundlemon

bundlemon Bot commented Jul 30, 2026 •

Copy link
Copy Markdown

BundleMon

No change in files bundle size

Groups updated (1)
Status Path Size Limits
✅ Build Folder
./**/*
5.52MB (+4.07KB +0.07%) +20%

Final result: ✅

View report in BundleMon website ➡️


Current branch size history | Target branch size history

@Ramon-Jimenez Ramon-Jimenez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @eperedo , working great.

Could you please make sure this fix, if needed, is also included on v42 branch?

Thanks.

@Ramon-Jimenez
Ramon-Jimenez merged commit bfc7ca3 into development Aug 5, 2026
11 checks passed
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.

3 participants