Uploaded image for project: 'SimplyE 2.0'
  1. SimplyE 2.0
  2. SIMPLY-2403

SimplyE incorrectly believes no server supports annotation sync

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Medium Medium
    • 6.6.3 (Android)
    • Android
    • None
    • S24 November 13 - November 27, SIMPLY S8 March 31 - April 14, SIMPLY S9 April 14 - 28, SIMPLY S10 April 28 - May 12, SIMPLY S11 May 12 - May 26, SIMPLY S12 May 26 - June 9, SIMPLY S13 June 9 - June 23, SIMPLY S14 June 23 - July 7, SIMPLY S15 July 7 - July 21, SIMPLY S16 July 21 - August 4, SIMPLY S17 August 4 - 18, SIMPLY S18 August 18 - Sep 1, SIMPLY S19 September 1 - 15, SIMPLY 22 Oct 14 - Oct 27, SIMPLY S20 September 15 - 29, SIMPLY S21 Sep 29 - Oct 13, SIMPLY S23 Oct 27 - Nov 10, SIMPLY S24 Nov 10 - 24, SIMPLY S25 Nov 24 - Dec 8, SIMPLY Sprint 26 Dec 8 - 22, SIMPLY S0 Dec 22 - Jan 5, SIMPLY S1 January 5 - 19, SIMPLY S2 Jan 19 - Feb 2, SIMPLY S3 Feb 2 - 16, SIMPLY S4 Feb 16 - March 2, SIMPLY S5 March 2 - March 16, SIMPLY S6 March 16 - March 30, SIMPLY S7 March 30 - April 13

      https://github.com/NYPL-Simplified/android/blob/develop/simplified-accounts-source-resolution/src/main/java/org/nypl/simplified/accounts/source/resolution/AccountProviderSourceStandardDescription.kt#L136 sets "supportsSimplyeSychronization" to false and there's no situation where it might be set to true. This basically disables the sync of current reading position and bookmarks across devices.

      This used to be an item in Accounts.json, and it looks like when we got rid of Account.json, the rules for determining the correct value (look in an OPDS feed for a link with rel="http://www.w3.org/ns/oa#annotationService") weren't implemented.

      There may be a similar problem in iOS, because the code changed on both platforms at around the same time. It's worth checking just to be safe. If iOS does work, then iOS is the place to look for the logic around determining whether the answer to this question is "really" yes or no.

            risawolf Risa Wolf
            leonardrichardson Leonard Richardson [X] (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: