Skip to content

feat: build asset notes, document attachments, maintenance records, and transfer workflows - #1243

Merged
yusuftomilola merged 1 commit into
DistinctCodes:mainfrom
abdulrcrtw:feature/abdulrcrtw-assigned-fixes
Jul 28, 2026
Merged

feat: build asset notes, document attachments, maintenance records, and transfer workflows#1243
yusuftomilola merged 1 commit into
DistinctCodes:mainfrom
abdulrcrtw:feature/abdulrcrtw-assigned-fixes

Conversation

@abdulrcrtw

Copy link
Copy Markdown
Contributor

Summary of Changes

This pull request implements the backend modules assigned to abdulrcrtw:

Closes #1145
Closes #1144
Closes #1143
Closes #1142

…nd transfer workflows

Implement modules assigned to abdulrcrtw.

Closes DistinctCodes#1145
Closes DistinctCodes#1144
Closes DistinctCodes#1143
Closes DistinctCodes#1142
@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

@abdulrcrtw is attempting to deploy a commit to the naijabuz's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jul 28, 2026

Copy link
Copy Markdown

@abdulrcrtw Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@yusuftomilola yusuftomilola left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed the diff — matches its description. Adds asset notes and document-attachment endpoints (notes-docs.controller.ts), a MaintenanceModule (scheduled/completed maintenance records), and a TransfersModule with approve/reject endpoints for asset transfer workflows. All new, distinct file paths — no collisions with previously merged assets/ files.

Same recurring app.module.ts additive-registration conflict as the last several PRs (new imports/module registrations landing at the same insertion point as prior merges); combined both since they're independent modules with no functional overlap. Resolved locally and pushed to main. Approving.

@yusuftomilola
yusuftomilola merged commit 439e993 into DistinctCodes:main Jul 28, 2026
2 of 9 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.

[BE-92] Asset transfers with an approval workflow [BE-91] Asset maintenance records [BE-90] Asset document attachments [BE-89] Asset notes

2 participants