-
Notifications
You must be signed in to change notification settings - Fork 21
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
feat: Add version audit record #567
Conversation
Codecov Report
@@ Coverage Diff @@
## master #567 +/- ##
==========================================
- Coverage 16.23% 16.22% -0.02%
==========================================
Files 98 98
Lines 1595 1596 +1
Branches 341 342 +1
==========================================
Hits 259 259
- Misses 1281 1282 +1
Partials 55 55
Continue to review full report at Codecov.
|
4ac2caf
to
8e7de15
Compare
const { appId } = match.params; | ||
|
||
await appVersionStore.fetchAll({ app_id: appId }); | ||
const versions = appVersionStore.versions; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
const {versions} = appVersionStore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok.
} | ||
} | ||
|
||
expandReason(audit) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rename the method to toggleReason
is better
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok.
No description provided.