Page MenuHomePhorge

Reseller comissioning system
ClosedPublic

Authored by machniak on Apr 9 2021, 2:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 12, 6:12 AM
Unknown Object (File)
Mar 17 2024, 10:33 AM
Unknown Object (File)
Feb 25 2024, 1:22 PM
Unknown Object (File)
Feb 19 2024, 8:23 AM
Unknown Object (File)
Feb 18 2024, 11:32 PM
Unknown Object (File)
Feb 8 2024, 2:55 PM
Unknown Object (File)
Feb 2 2024, 6:40 PM
Unknown Object (File)
Jan 29 2024, 6:24 AM
Subscribers
Restricted Project

Details

Reviewers
vanmeeuwen
Group Reviewers
Restricted Project
Commits
rK2de8edd9f18b: Reseller comissioning system

Diff Detail

Repository
rK kolab
Branch
dev/reseller-comissioning
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 34132
Build 13588: arc lint + arc unit

Event Timeline

machniak created this revision.
machniak retitled this revision from Reseller comissioning system to [WIP] Reseller comissioning system.May 12 2021, 11:37 AM
machniak edited the summary of this revision. (Show Details)
  • Migration fixes, tests fixes, added some simple tests
  • Add tests
  • Debit reseller's wallet on user deletion (for the negative balance value)
  • Credit/debit reseller's wallet on Penalize/Award action
machniak retitled this revision from [WIP] Reseller comissioning system to Reseller comissioning system.May 13 2021, 3:33 PM
machniak edited the summary of this revision. (Show Details)
vanmeeuwen subscribed.

Didn't go through the tests yet.

src/app/Http/Controllers/API/V4/Admin/WalletsController.php
103 ↗(On Diff #7090)

Perhaps not the wording we'll want to be using under each of the circumstances that create a positive or negative charge.

We should elaborate during the next standup perhaps.

src/app/PackageSku.php
44 ↗(On Diff #7090)

It was implemented as a potential future feature, where despite the sku costs and free units, the package cost could still be set separately.

src/app/Providers/PaymentProvider.php
264

At the very least there's a spelling error in the word "commission", but since its also a FIXME, that's not even what I seek to point out.

This revision now requires changes to proceed.May 14 2021, 5:20 PM
This revision is now accepted and ready to land.May 18 2021, 3:12 PM