Page MenuHomePhorge

Transaction log in admin UI
ClosedPublic

Authored by machniak on Jun 10 2020, 8:14 AM.
Tags
None
Referenced Files
F11585394: D1360.id3583.diff
Thu, Mar 28, 11:41 AM
Unknown Object (File)
Sun, Mar 24, 9:17 PM
Unknown Object (File)
Sun, Mar 17, 2:43 PM
Unknown Object (File)
Sun, Mar 17, 12:26 PM
Unknown Object (File)
Sun, Mar 17, 7:20 AM
Unknown Object (File)
Sat, Mar 2, 9:29 AM
Unknown Object (File)
Feb 22 2024, 11:24 PM
Unknown Object (File)
Feb 8 2024, 7:06 AM
Subscribers
Restricted Project

Details

Reviewers
vanmeeuwen
Group Reviewers
Restricted Project
Commits
rK90da82738276: Transaction log in admin UI
Summary

Created TransactionLog widget, which is now used in User and Admin apps.
The transaction log table for admins displays additonal User column.

Test Plan

./phpunit

Diff Detail

Repository
rK kolab
Branch
dev/wallet-history-support
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 29764
Build 11014: arc lint + arc unit

Event Timeline

machniak created this revision.
bohlender added inline comments.
src/resources/vue/Admin/User.vue
149

shouldn't the title be conditionally rendered as well ?

src/resources/vue/Domain/Info.vue
3

Are we going to use the shorthand for v-bind (and v-on) exclusively ?

src/resources/vue/Admin/User.vue
149

The v-if here is not to not render the table, but to render it when it is ready. So, it will always be there.

src/resources/vue/Domain/Info.vue
3

This is what our current linter suggests. npm run lint.

This revision is now accepted and ready to land.Jun 10 2020, 11:32 AM
This revision was automatically updated to reflect the committed changes.