-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
SF-2860 Fix offline project add question error #2966
Conversation
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.
Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion
src/SIL.XForge.Scripture/ClientApp/src/app/checking/question-dialog/question-dialog.service.ts
line 108 at r1 (raw file):
private async canCreateAndEditQuestions(projectId: string): Promise<boolean> { const userId = this.userService.currentUserId; const project = (await this.projectService.getProfile(projectId)).data;
I'm not sure about this change. This seemed to work for retrieving a project profile doc, where get(projectId)
to get a project doc would not when offline. Could this be because the project profile docs were loaded before I went offline on the project connect page?
188fddf
to
9a75a12
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2966 +/- ##
=======================================
Coverage 82.08% 82.08%
=======================================
Files 544 544
Lines 31637 31637
Branches 5147 5142 -5
=======================================
Hits 25969 25969
Misses 4901 4901
Partials 767 767 ☔ View full report in Codecov by Sentry. |
Please update the title to conform to the guidelines in CONTRIBUTING.md. |
Thanks @siltomato |
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.
Reviewed 2 of 2 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @siltomato)
src/SIL.XForge.Scripture/ClientApp/src/app/checking/question-dialog/question-dialog.service.ts
line 108 at r1 (raw file):
Previously, siltomato wrote…
I'm not sure about this change. This seemed to work for retrieving a project profile doc, where
get(projectId)
to get a project doc would not when offline. Could this be because the project profile docs were loaded before I went offline on the project connect page?
Yes, that will be it. Usually we load the project profile rather than the project, as you only really need the project doc if you want the paratextUsers property.
9a75a12
to
4c2eeb0
Compare
4c2eeb0
to
f49f42a
Compare
f49f42a
to
7259515
Compare
Fixes issue where adding a question with empty xforge indexeddb database would throw an error (see Jira for video capture).
This change is