-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[PM-8254] Create shareable export component #9246
[PM-8254] Create shareable export component #9246
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #9246 +/- ##
=======================================
Coverage 28.15% 28.15%
=======================================
Files 2375 2375
Lines 70222 70222
Branches 13173 13173
=======================================
Hits 19773 19773
Misses 48884 48884
Partials 1565 1565 ☔ View full report in Codecov by Sentry. |
No New Or Fixed Issues Found |
Fix lint issue with takeUntil
Copied existing export.component.html as that has already been migrated to use the component library components Strip the markup from the dialog and the submit-button
…mit, loading, disabled) Emit successful Export Expose a form-id so the hosting component can bind to this form Fix name of output
…dVault is collected
1423391
to
1378eb4
Compare
libs/tools/export/vault-export/vault-export-ui/src/components/export.component.ts
Outdated
Show resolved
Hide resolved
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.
🎨 suggestion
⛏️ nitpick
🤔 non-blocking concern
libs/tools/export/vault-export/vault-export-ui/src/components/export.component.ts
Outdated
Show resolved
Hide resolved
this.bitSubmit.loading$.pipe(takeUntil(this.destroy$)).subscribe((loading) => { | ||
this.formLoading.emit(loading); | ||
}); | ||
|
||
this.bitSubmit.disabled$.pipe(takeUntil(this.destroy$)).subscribe((disabled) => { | ||
this.formDisabled.emit(disabled); | ||
}); | ||
} |
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.
🎨 EventEmitter<T>
implements Subject<T>
. You should be able to subscribe to it directly:
this.bitSubmit.loading$
.pipe(takeUntil(this.destroy$))
.subscribe(this.formLoading);
this.bitSubmit.disabled$
.pipe(takeUntil(this.destroy$))
.subscribe(this.formDisabled);
Type of change
Objective
Code changes
Before you submit