-
-
Notifications
You must be signed in to change notification settings - Fork 650
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(sheets): add watch API to ref-range-service #1635
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #1635 +/- ##
==========================================
+ Coverage 29.90% 29.96% +0.06%
==========================================
Files 1162 1162
Lines 62963 63095 +132
Branches 13232 13268 +36
==========================================
+ Hits 18826 18909 +83
- Misses 44137 44186 +49 ☔ View full report in Codecov by Sentry. |
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.
Some issue need to be addressed. @ybzky
@@ -0,0 +1,435 @@ | |||
/** |
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.
Be careful with the file's name. It should be in kebab-case.
@@ -64,6 +103,38 @@ export class RefRangeService extends Disposable { | |||
}); | |||
} | |||
|
|||
watchRange(unitId: string, subUnitId: string, range: IRange, callback: WatchRangeCallback): IDisposable { |
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.
This API seems not tested.
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.
Some issue need to be addressed. @ybzky
f0b3180
to
0ad7202
Compare
View Deployment
|
2262545
to
10eda64
Compare
close #xxx, #yyy, #zzzz