Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update bill manager to latest api docs and use derive builder. #92

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

Update bill manager to latest api docs and use derive builder. #92

wants to merge 8 commits into from

Conversation

crispinkoech
Copy link
Collaborator

Closes #87 and #91; still work in progress.

@c12i
Copy link
Owner

c12i commented Nov 29, 2023

Hey @crispinkoech should I review while still in draft?

@crispinkoech
Copy link
Collaborator Author

crispinkoech commented Nov 29, 2023 via email

@crispinkoech
Copy link
Collaborator Author

Oooft, this MR is too out of sync with the current master commit; @itsyaasir @c12i I suggest I close this and start some new work off of master?

Also, when was the cancel_bulk_invoice.rs module/functionality removed? Can't seem to find a merge request that deleted it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@itsyaasir itsyaasir Awaiting requested review from itsyaasir

@c12i c12i Awaiting requested review from c12i

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update Bill manager module to use derive_builder macro.
2 participants
@crispinkoech @c12i