search

lbryio / lbry-desktop

A browser and wallet for LBRY, the decentralized, user-controlled content marketplace.
https://lbry.tech
MIT License
3.57k stars 415 forks source link

unable to set preferences from old preference format on lbry.tv/desktop #4714

Closed tzarebczan closed 4 years ago

tzarebczan commented 4 years ago

Users who haven't upgraded to new settings format will see this error and lose all settings. The preference get call does return the correct old style settings (below), but nothing is set. Not sure when this broke, but it used to work when we rolled out with it. 


{
  "jsonrpc": "2.0",
  "result": {
    "shared": {
      "type": "object",
      "value": {
        "blocked": [],
        "subscriptions": [
          "lbry://@emmyhucker#4ad942982e43326c7700b1b6443049b3cfd82161",
          "lbry://@OYENEWS#e11e2fc3056137948d2cc83fb5ca2ce9b57025ec",
          "lbry://@LBRYnomics#2d09719c8e06ab54ca5c1b4e44ddf3ee9d30241f",
          "lbry://@MH#5bd299a92e7b31865d2bb3e2313402edaca41a94",
          "lbry://@RichardHarrisCoaching#4506db7fb52d3ec5d3a024c870bf86fc35f7b6a3",
          "lbry://@Truce#0d5ff2c7d8c064a8de9ee8335f1864303ab4710b",
          "lbry://@HackerSploit#26c9b54d7e47dc8f7dc847821b26fce3009ee1a0",
          "lbry://@lbry#3fda836a92faaceedfe398225fb9b2ee2ed1f01a",
          "lbry://@MinuteVideos#46be492ee0f56db11e005991c537c867a8682f77",
          "lbry://@Wranglerstar#4b602d7a3e268abb45951f623a109d2a131ab0ba",
          "lbry://@lafenixtv#5445c3902df0ead971531aa3ee8c9e50c6bb67b0",
          "lbry://@mime#649f5d766f6ca2db1be249b09e164ffe52939751",
          "lbry://@eevblog#74333143a3dcc001a5602aa524583fc75a013d75",
          "lbry://@davidpakman#760da3ba3dd85830a843beaaed543a89b7a367e7",
          "lbry://@canalvocenaosabia#8388e7b9d096d0762d220bf50f37158667a84d2f",
          "lbry://@Edu-Primitivo#91e156f1dfc0bdec09539613578da692587e4ea9",
          "lbry://@SpikeViper#930fc43ca7bae20d4706543e97175d1872b0671f",
          "lbry://@Kazzio#a55bf149420fa4b16e2f7dac055093438eb5c7aa",
          "lbry://@Top-LBRY-YouTubers#dae9f6da69a81f157ba2948fca9e566ecbe09cf0",
          "lbry://@LBRYClass#f8d6eccd887c9cebd36b1d42aa349279b7f5c3ed"
        ],
        "tags": [
          "blockchain",
          "economics",
          "education",
          "music",
          "news",
          "science"
        ]
      },
      "version": "0.1"
    }
  },
  "id": 0
}```
tzarebczan commented 4 years ago

fixed in https://github.com/lbryio/lbry-desktop/pull/4718, thanks @jessopb !