-
Notifications
You must be signed in to change notification settings - Fork 16
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
Draft Design #39
base: main
Are you sure you want to change the base?
Draft Design #39
Conversation
Hi @drakkomaximo thanks for your contribution! Please make sure to have the option to configure the clear and dark mode. Also, it's important that you mantein the side bar when the lender is on the audit page. This could be an example of table: Also, you can take into account this is the Audit model: Please remember to use SHADCN UI and Lucide React in the design. I will share with you a code of reference! Here are some visual indicators like guide:
Here is a specific information about any details of the table:
Table Columns
|
I need to move this from here src\app\lender\audit-logs\page.tsx to here src\app\admin Also delete this from src\app\api\audit\logs\route.ts because it doesn't go here, that should go here src@types but only the file and rename it audits.entity.ts And finally move this src\components\audit-logs-table.tsx to here src\components\admin in a folder called audit-logs |
Alright, I'll be making the changes throughout the day tomorrow. If anything comes up, you can message me on Telegram for faster communication. |
Hi @JosueBrenes In that route: src\app\api\audit\logs\route.ts, I'm simulating as if the information were coming through an API. I can move part of the data, but the core should remain, which is the GET request, to keep the path ready for future backend integration. |
Hi @JosueBrenes Done! 2025-02-24.01-40-14.mp42025-02-24.02-02-05.mp42025-02-24.02-45-07.mp4I was reading the project requirements, and they specify certain routes, which is why I created the folders from the beginning. In this case, who should I follow? |
Hi @drakkomaximo the table is really good. Please remember to incluye a search bar and a filter, but it's almost ready! Thanks |
Hi @gchyuli, but in the evidence, I'm showing that the search bar and filters are applied to the table. Or are you talking about a different topic? 2025-02-24.02-02-05.mp4 |
Pull Request for TrustBridge - Close Issue #33
❗ Pull Request Information
Implementation of an Audit Logs page in the lender's dashboard that allows users with the lender role to view and track platform actions with a modern, animated interface.
🌀 Summary of Changes
New Components:
AuditLogsTable
component with Framer Motion animationsAPI Implementation:
/api/audit/logs
endpoint with pagination supportUI/UX Improvements:
Features:
🛠 Testing
Evidence Before Solution
Evidence After Solution
/lender/audit-logs
📂 Related Issue
This pull request will close #33 upon merging.
Technical Details
Files Changed
src/components/audit-logs-table.tsx
src/app/lender/audit-logs/page.tsx
src/app/api/audit/logs/route.ts
Evidence
2025-02-23.13-51-58.mp4