Opened 7 years ago

Closed 3 years ago

#6488 closed defect (fixed)

"keep"ing a read session throws an error; need a way to bookmark

Reported by: sj Owned by: joe
Priority: high Milestone: 8.2.0 (was Update.2)
Component: read-activity Version: Build 650
Keywords: Read-50:+ joyride-2301:+ Cc: morgs
Blocked By: Blocking:
Deployments affected: Action Needed: qa signoff
Verified: no

Description

keeping a read session throws an error. this should ideally provide something like a bookmark for a book; at least a journal entry that can be resumed at that page. In this case, the journal entry in question should pull out and display (in the journal's metadata) the page at which it was 'kept'.

Reported by dpbsmith.

Attachments (1)

0001-6488-Fix-keep-error-workaround-for-now-will-fix.patch (1.1 KB) - added by morgs 6 years ago.
Workaround pushed to git, will be in next release

Download all attachments as: .zip

Change History (12)

comment:1 Changed 7 years ago by sj

  • Version set to Build 650

comment:2 Changed 6 years ago by gregorio

  • Milestone Never Assigned deleted

Milestone Never Assigned deleted

comment:3 Changed 6 years ago by marco

  • Action Needed set to diagnose
  • Milestone set to 9.1.0

comment:4 Changed 6 years ago by mstone

  • Keywords 8.2.0:- 9.1.0:? added
  • Priority changed from normal to high

This is a major usability issue for Read. If we find a Read maintainer, fixing this should be a priority.

comment:5 Changed 6 years ago by morgs

  • Owner changed from rwh to morgs

comment:6 Changed 6 years ago by morgs

See traceback in #7843

Changed 6 years ago by morgs

Workaround pushed to git, will be in next release

comment:7 Changed 6 years ago by morgs

  • Action Needed changed from diagnose to add to build
  • Status changed from new to assigned

Workaround pushed to git.

comment:8 follow-up: Changed 6 years ago by morgs

This should fix #6982 as well.

comment:9 in reply to: ↑ 8 Changed 6 years ago by morgs

Replying to morgs:

This should fix #6982 as well.

Actually not, because the metadata doesn't survive reboots - #4662

comment:10 Changed 6 years ago by morgs

  • Action Needed changed from add to build to qa signoff
  • Cc morgs added
  • Keywords Read-50:+ joyride-2301:+ added; 8.2.0:- 9.1.0:? removed
  • Milestone changed from 9.1.0 to 8.2.0 (was Update.2)
  • Owner changed from morgs to joe
  • Status changed from assigned to new

Fixed in Read-50. Tested on joyride-2301.

Test case: Open a PDF on XO A (e.g. from the examples available in the library through Browse's homepage). Stop the activity. No errors will be displayed. The journal entry will be updated appropriately.

comment:11 Changed 3 years ago by godiard

  • Resolution set to fixed
  • Status changed from new to closed

This ticket was fixed 3 years ago.

Note: See TracTickets for help on using tickets.