{
  "founder": "meshcollider",
  "channel": "#bitcoin-core-dev",
  "network": "freenode",
  "id": "a27a87507bcd4e71b1cb8cd837af3eef",
  "name": "#bitcoin-core-dev",
  "chair": "meshcollider",
  "chairs": [
    "meshcollider"
  ],
  "nicks": {
    "meshcollider": 32,
    "lightningbot": 2,
    "jnewbery": 44,
    "achow101": 38,
    "gribble": 7,
    "kanzure": 6,
    "luke-jr": 3,
    "harding": 3,
    "ryanofsky": 14,
    "sipa": 2,
    "bitcoin-git": 1,
    "provoostenator": 11
  },
  "start_time": "2019-04-12T19:00:04+00:00",
  "end_time": "2019-04-12T19:56:16+00:00",
  "active": false,
  "original_topic": "Bitcoin Core development discussion and commit log | This is the channel for developing Bitcoin Core. Feel free to watch, but please take commentary and usage questions to #bitcoin | Channel logs: http://www.erisian.com.au/bitcoin-core-dev/, http://gnusha.org/bitcoin-core-dev/ | Meeting topics https://gist.github.com/moneyball/071d608fdae217c2a6d7c35955881d8a",
  "current_topic": "native descriptor wallets (achow101)",
  "messages": [
    {
      "id": "62e539ee63444e829a2e73d6f6b4b4e8",
      "sender": "meshcollider",
      "payload": "#startmeeting",
      "action": false,
      "timestamp": "2019-04-12T19:00:04+00:00"
    },
    {
      "id": "26c9870869b84c7ab271d9068031c409",
      "sender": "lightningbot",
      "payload": "Meeting started Fri Apr 12 19:00:04 2019 UTC.  The chair is meshcollider. Information about MeetBot at http://wiki.debian.org/MeetBot.",
      "action": false,
      "timestamp": "2019-04-12T19:00:04+00:00"
    },
    {
      "id": "6d855539d73e46b08dd8b323337a75bc",
      "sender": "lightningbot",
      "payload": "Useful Commands: #action #agreed #help #info #idea #link #topic.",
      "action": false,
      "timestamp": "2019-04-12T19:00:04+00:00"
    },
    {
      "id": "d23db141da3c4d6e9aec0c7d48115089",
      "sender": "meshcollider",
      "payload": "#bitcoin-core-dev Wallet Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier jl2012 achow101 meshcollider jnewbery maaku fanquake promag provoostenator aj Chris_Stewart_5 dongcarl gwillen jamesob ken281221 ryanofsky gleb",
      "action": false,
      "timestamp": "2019-04-12T19:00:07+00:00"
    },
    {
      "id": "ab8aa5009e164cfda3633888c8f3fd93",
      "sender": "jnewbery",
      "payload": "hi",
      "action": false,
      "timestamp": "2019-04-12T19:00:11+00:00"
    },
    {
      "id": "517375d94bec4472bf259d00f9a586a6",
      "sender": "achow101",
      "payload": "hi",
      "action": false,
      "timestamp": "2019-04-12T19:01:10+00:00"
    },
    {
      "id": "6584f639ad9e46bda5ecebdad4dc46b9",
      "sender": "meshcollider",
      "payload": "#topic : Upgrade wallet (#15761) (jnewbery)",
      "action": false,
      "timestamp": "2019-04-12T19:01:35+00:00"
    },
    {
      "id": "cdb8a1efd95249d9a03ba15451178da3",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/15761 | Replace -upgradewallet startup option with upgradewallet RPC by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #15761 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2019-04-12T19:01:37+00:00"
    },
    {
      "id": "0d6aabfde9f94aa1aaebc972eab9b71d",
      "sender": "kanzure",
      "payload": "hi",
      "action": false,
      "timestamp": "2019-04-12T19:02:14+00:00"
    },
    {
      "id": "dfcfac4cb05d4f2cb0d03514d0fce88b",
      "sender": "jnewbery",
      "payload": "This PR is to replace the -upgradewallet startup option, which is great news",
      "action": false,
      "timestamp": "2019-04-12T19:02:26+00:00"
    },
    {
      "id": "2323973c88a7467ca47223d6ce22a2da",
      "sender": "jnewbery",
      "payload": "achow is proposing to replace it with an RPC. There were a couple of other suggestions on what to replace it with in the PR and I wanted to poll for opinions",
      "action": false,
      "timestamp": "2019-04-12T19:03:01+00:00"
    },
    {
      "id": "ea23c5df6e514e7298ee01d5d9360637",
      "sender": "achow101",
      "payload": "I think the primary question is RPC, wallet-tool command, or both",
      "action": false,
      "timestamp": "2019-04-12T19:03:10+00:00"
    },
    {
      "id": "1ec9873221414a48a12ad07c319b0758",
      "sender": "achow101",
      "payload": "I prefer both",
      "action": false,
      "timestamp": "2019-04-12T19:03:30+00:00"
    },
    {
      "id": "69c894b7490241d292b2f1baed4193a5",
      "sender": "jnewbery",
      "payload": "I'm slightly concerned about having an RPC that can upgrade at any time. I just feel that it might introduce subtle corner cases if the wallet is doing something else at the time",
      "action": false,
      "timestamp": "2019-04-12T19:03:47+00:00"
    },
    {
      "id": "de3deb86bf26430d9c5588f22ab9519a",
      "sender": "jnewbery",
      "payload": "but I might be wrong. Perhaps it's fine",
      "action": false,
      "timestamp": "2019-04-12T19:04:16+00:00"
    },
    {
      "id": "76fb783ddf9e4f42bbc926083d54dd4d",
      "sender": "achow101",
      "payload": "the whole operation is locked, and any operations on the wallet are also locked, so I don't think that's really a problem",
      "action": false,
      "timestamp": "2019-04-12T19:04:44+00:00"
    },
    {
      "id": "52ad9116b75b423d80ca7dad3024490b",
      "sender": "jnewbery",
      "payload": "(I'm definitely a big concept ACK on moving away from startup option)",
      "action": false,
      "timestamp": "2019-04-12T19:04:47+00:00"
    },
    {
      "id": "af809ad200ed4e34a16b1b923005e7ef",
      "sender": "kanzure",
      "payload": "does upgradewallet replace the wallet or does it create an upgraded wallet only?",
      "action": false,
      "timestamp": "2019-04-12T19:05:28+00:00"
    },
    {
      "id": "7d00083c402c4d13b8a73d355597e978",
      "sender": "achow101",
      "payload": "kanzure: it just upgrades the wallet file",
      "action": false,
      "timestamp": "2019-04-12T19:05:40+00:00"
    },
    {
      "id": "45edd93c9e5f4b148788803b8ae9f9b1",
      "sender": "luke-jr",
      "payload": "could be only allowed when loading",
      "action": false,
      "timestamp": "2019-04-12T19:05:52+00:00"
    },
    {
      "id": "f246e2fc96a0443f8e401e18f7dc7e67",
      "sender": "meshcollider",
      "payload": "I'm in favour of both too",
      "action": false,
      "timestamp": "2019-04-12T19:06:03+00:00"
    },
    {
      "id": "78486ebe1b1b478a87a6822297395fd3",
      "sender": "jnewbery",
      "payload": "luke-jr: that's one suggestion in the PR. Another option would be to only allow it on wallets that aren't currently loaded.",
      "action": false,
      "timestamp": "2019-04-12T19:06:50+00:00"
    },
    {
      "id": "5e814e05be1946f58adb18ad91aaa1bb",
      "sender": "jnewbery",
      "payload": "(ie separate upgrade from running entirely)",
      "action": false,
      "timestamp": "2019-04-12T19:07:11+00:00"
    },
    {
      "id": "52f5d9e20cf444de83aa2f9cbd85ffaf",
      "sender": "achow101",
      "payload": "jnewbery: what if upgradewallet unloads it, upgrades, then reloads it",
      "action": false,
      "timestamp": "2019-04-12T19:07:13+00:00"
    },
    {
      "id": "eefc11d821b5459491f00dd95c5d6ec5",
      "sender": "achow101",
      "payload": "that would disconnect all of the signals and stuff that would cause issues",
      "action": false,
      "timestamp": "2019-04-12T19:07:29+00:00"
    },
    {
      "id": "f1ae8fd3b8624acaab21129662cc29f8",
      "sender": "kanzure",
      "payload": "need lock during upgrade",
      "action": false,
      "timestamp": "2019-04-12T19:07:34+00:00"
    },
    {
      "id": "e12cb3127d814b7894cb874bab541239",
      "sender": "kanzure",
      "payload": "so that they don't reopen",
      "action": false,
      "timestamp": "2019-04-12T19:07:38+00:00"
    },
    {
      "id": "0a28f47f37c94d8db4cfbccbe20f675d",
      "sender": "kanzure",
      "payload": "in achow101's flow.",
      "action": false,
      "timestamp": "2019-04-12T19:07:52+00:00"
    },
    {
      "id": "aff645bf032d4e46b13085cbf7d0053f",
      "sender": "jnewbery",
      "payload": "Should we encourage users to backup before they upgrade?",
      "action": false,
      "timestamp": "2019-04-12T19:08:16+00:00"
    },
    {
      "id": "f7742ebdba994e028528e118e2dcae4f",
      "sender": "meshcollider",
      "payload": "kanzure: I think thats the current behavior anyway",
      "action": false,
      "timestamp": "2019-04-12T19:08:30+00:00"
    },
    {
      "id": "f72548f07dad4ca7945656d2a79d9ec6",
      "sender": "kanzure",
      "payload": "meshcollider: ok, i thought the lock only applies to loaded wallets. nevermind.",
      "action": false,
      "timestamp": "2019-04-12T19:08:53+00:00"
    },
    {
      "id": "15c2eb79a8a44bdca35f3adf6a817cf2",
      "sender": "achow101",
      "payload": "kanzure: meshcollider: in the idea I just proposed, you would have to lock something to prevent loading the wallet during upgarde",
      "action": false,
      "timestamp": "2019-04-12T19:09:29+00:00"
    },
    {
      "id": "289db3652fa4476c9cbe373ae8da9b3b",
      "sender": "luke-jr",
      "payload": "jnewbery: at least after",
      "action": false,
      "timestamp": "2019-04-12T19:09:35+00:00"
    },
    {
      "id": "c37d3212c6574c50b0dc4a1504d8aca8",
      "sender": "harding",
      "payload": "Wallet files are generally small.  If you think a backup is important, it's probably better to just make one automatically and stuff it somewhere in ~/.bitcoin/",
      "action": false,
      "timestamp": "2019-04-12T19:09:58+00:00"
    },
    {
      "id": "eb16a96c775441aa9e63f32ede4cacb5",
      "sender": "achow101",
      "payload": "jnewbery: probably. the help text for the RPC does say that backups after are required",
      "action": false,
      "timestamp": "2019-04-12T19:10:07+00:00"
    },
    {
      "id": "77d436a13f284b5aa01ad718305755fc",
      "sender": "achow101",
      "payload": "I can change it to also say before too",
      "action": false,
      "timestamp": "2019-04-12T19:10:15+00:00"
    },
    {
      "id": "5e3c7dccb5ed4abf8971a739b1f4d92b",
      "sender": "meshcollider",
      "payload": "backup before can be done automatically like harding suggests",
      "action": false,
      "timestamp": "2019-04-12T19:10:32+00:00"
    },
    {
      "id": "b358c084d61540f78978cfd8e7e78b25",
      "sender": "meshcollider",
      "payload": "Make a wallet.old or something",
      "action": false,
      "timestamp": "2019-04-12T19:10:41+00:00"
    },
    {
      "id": "e01cdcb52c3944a9874868f6ef6ed504",
      "sender": "harding",
      "payload": "(Backup after is a different thing, since that's talking about off-disk backups.)",
      "action": false,
      "timestamp": "2019-04-12T19:10:47+00:00"
    },
    {
      "id": "10f057d42b5146dab323edd5ff4747f7",
      "sender": "luke-jr",
      "payload": "achow101: well, only after is *required*..",
      "action": false,
      "timestamp": "2019-04-12T19:11:14+00:00"
    },
    {
      "id": "b019e56e1343473296bf06165b662274",
      "sender": "achow101",
      "payload": "right",
      "action": false,
      "timestamp": "2019-04-12T19:12:58+00:00"
    },
    {
      "id": "4cf31c5308df434db7ac81f3f881ef75",
      "sender": "achow101",
      "payload": "I think it's possible to make both an upgradewallet RPC and wallet-tool command work safely",
      "action": false,
      "timestamp": "2019-04-12T19:14:17+00:00"
    },
    {
      "id": "f5ed6513a1f44900b0803ac066cd00cc",
      "sender": "meshcollider",
      "payload": "It doesn't seem like theres any real objection to that",
      "action": false,
      "timestamp": "2019-04-12T19:15:58+00:00"
    },
    {
      "id": "d6f71cac2dbb4906afc475803400e95a",
      "sender": "meshcollider",
      "payload": "jnewbery: anything else to discuss on this topic?",
      "action": false,
      "timestamp": "2019-04-12T19:16:56+00:00"
    },
    {
      "id": "0abe063f94dd4ccd963a650e31cf3e0f",
      "sender": "jnewbery",
      "payload": "nothing from me",
      "action": false,
      "timestamp": "2019-04-12T19:17:12+00:00"
    },
    {
      "id": "a579447284584824bc35309ea5eeb06b",
      "sender": "meshcollider",
      "payload": "#topic native descriptor wallets (achow101)",
      "action": false,
      "timestamp": "2019-04-12T19:17:31+00:00"
    },
    {
      "id": "066c7864ee314c6e82006dfeb575f74b",
      "sender": "achow101",
      "payload": "So i finished up an implementation of native descriptor wallets last weekend: #15764",
      "action": false,
      "timestamp": "2019-04-12T19:18:01+00:00"
    },
    {
      "id": "35a4b37d9051458984d96c310718e44d",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/15764 | Native descriptor wallets by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #15764 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2019-04-12T19:18:03+00:00"
    },
    {
      "id": "a800e7d97dc6470eacc05ceb2cc92e8c",
      "sender": "achow101",
      "payload": "the only question I really have is what to do with the tests for the old wallet stuff",
      "action": false,
      "timestamp": "2019-04-12T19:18:26+00:00"
    },
    {
      "id": "b7a7103b68bf46e7b0003dfc2ce88d9b",
      "sender": "jnewbery",
      "payload": "I think it's pretty important to keep those tests",
      "action": false,
      "timestamp": "2019-04-12T19:18:50+00:00"
    },
    {
      "id": "648829b953164ef2a621a876e7f356b6",
      "sender": "achow101",
      "payload": "currently I have a bunch of tests removed or changed because they don't work with descriptor wallets",
      "action": false,
      "timestamp": "2019-04-12T19:18:58+00:00"
    },
    {
      "id": "ea90fc030cb541e29df164ec17f4c821",
      "sender": "jnewbery",
      "payload": "Most users will still be using non-descriptor wallets for some time. We can't just stop testing those and hope there are no regressions",
      "action": false,
      "timestamp": "2019-04-12T19:19:17+00:00"
    },
    {
      "id": "3450cb609a8142a1af923ecb54e75aa7",
      "sender": "ryanofsky",
      "payload": "you can add descriptors as an optional feature, there's no need to remove old code or old tests",
      "action": false,
      "timestamp": "2019-04-12T19:19:45+00:00"
    },
    {
      "id": "8b5588c22c63403e8c4efaf49ef79850",
      "sender": "achow101",
      "payload": "ryanofsky: that's an option, but I feel like descriptors and its new definitions is such a departure from current wallet stuff that it should have better distinction than just a wallet flag",
      "action": false,
      "timestamp": "2019-04-12T19:22:31+00:00"
    },
    {
      "id": "a98946d03018431bb21b3180eed20c83",
      "sender": "ryanofsky",
      "payload": "you are referring to the practical downside of having to keep more code around?",
      "action": false,
      "timestamp": "2019-04-12T19:23:57+00:00"
    },
    {
      "id": "8bc263a9ff88470e8b6e6759d540f580",
      "sender": "achow101",
      "payload": "e.g. it's possible for someone to write code which accidentally unsets the wallet flag. it's much harder to accidentally downgrade the version",
      "action": false,
      "timestamp": "2019-04-12T19:23:58+00:00"
    },
    {
      "id": "79289aafb7224496b320cf140fbea761",
      "sender": "meshcollider",
      "payload": "Does your PR remove the ability to generate new \"old\" wallets",
      "action": false,
      "timestamp": "2019-04-12T19:24:01+00:00"
    },
    {
      "id": "c68f3c4194794f62b4d0c576aef2d713",
      "sender": "achow101",
      "payload": "meshcollider: yes",
      "action": false,
      "timestamp": "2019-04-12T19:24:15+00:00"
    },
    {
      "id": "f7e6132d66f44e8a8d28c057f901e3d2",
      "sender": "achow101",
      "payload": "(it's a wallet version bump)",
      "action": false,
      "timestamp": "2019-04-12T19:24:22+00:00"
    },
    {
      "id": "8a8c034de34e407ea1aa4806669ac8f5",
      "sender": "sipa",
      "payload": "achow101: i think, even for just testing purposes, we'll need to retain the ability to create old wallets",
      "action": false,
      "timestamp": "2019-04-12T19:26:02+00:00"
    },
    {
      "id": "f8a3f16f42be49ed9635d155834b5d42",
      "sender": "jnewbery",
      "payload": "The main downside of it being a flag rather than a version is that it becomes combinatorially more difficult to test everything",
      "action": false,
      "timestamp": "2019-04-12T19:26:15+00:00"
    },
    {
      "id": "99782a59453b413ab637ae499321a285",
      "sender": "sipa",
      "payload": "as this is a very invasive change, and i don't think we want to lose the ability to test old logic",
      "action": false,
      "timestamp": "2019-04-12T19:26:24+00:00"
    },
    {
      "id": "2c0162fc6a1b4f08bc57de2f91673d3a",
      "sender": "jnewbery",
      "payload": "I think that the default should be that newly created wallets are old-style, and users need to explicitly upgrade",
      "action": false,
      "timestamp": "2019-04-12T19:26:51+00:00"
    },
    {
      "id": "c4b459714aee464eade564ab9c1d3e7b",
      "sender": "bitcoin-git",
      "payload": "[bitcoin] jamesob opened pull request #15809: gitignore: plist and dat (master...2019-04-gitignore) https://github.com/bitcoin/bitcoin/pull/15809",
      "action": false,
      "timestamp": "2019-04-12T19:26:51+00:00"
    },
    {
      "id": "0e2121ed70354408a8a6637768fad2e6",
      "sender": "achow101",
      "payload": "jnewbery: how come?",
      "action": false,
      "timestamp": "2019-04-12T19:27:09+00:00"
    },
    {
      "id": "bbdde8a96f1448a2846064614c8ab676",
      "sender": "achow101",
      "payload": "I think an explicit upgrade is actually far more dangerous than creating a new descriptor wallet",
      "action": false,
      "timestamp": "2019-04-12T19:27:45+00:00"
    },
    {
      "id": "4723457610244019a7ac5f0d7da68a1b",
      "sender": "jnewbery",
      "payload": "because like you say, it's a big departure from the current wallet design",
      "action": false,
      "timestamp": "2019-04-12T19:27:45+00:00"
    },
    {
      "id": "169ea34dc0274faca4899a577086d1aa",
      "sender": "jnewbery",
      "payload": "we need to support both for some time to come, so why not take the conservative approach for now",
      "action": false,
      "timestamp": "2019-04-12T19:28:10+00:00"
    },
    {
      "id": "4db37f501fdf4029b5e47855c4af511a",
      "sender": "meshcollider",
      "payload": "I think thats sensible for now, old wallet generation can be \"deprecated\" later on",
      "action": false,
      "timestamp": "2019-04-12T19:28:21+00:00"
    },
    {
      "id": "2ffbbfe2368e44df979784af50584885",
      "sender": "ryanofsky",
      "payload": "yeah, i just think you don't need to add descriptors as this big one time change",
      "action": false,
      "timestamp": "2019-04-12T19:29:17+00:00"
    },
    {
      "id": "e42258a6a9074071b499203c00ea73b8",
      "sender": "jnewbery",
      "payload": "> I think an explicit upgrade is actually far more dangerous...",
      "action": false,
      "timestamp": "2019-04-12T19:29:50+00:00"
    },
    {
      "id": "2088699902cb42bebe3c3f55d850250e",
      "sender": "ryanofsky",
      "payload": "you can add new functionality alongside existing functionality, you will get better review better testing",
      "action": false,
      "timestamp": "2019-04-12T19:29:51+00:00"
    },
    {
      "id": "e724045d58ff4d89b85bacebd3a9de16",
      "sender": "jnewbery",
      "payload": "We definitely shouldn't be releasing wallet code that we think is *in any way* dangerous",
      "action": false,
      "timestamp": "2019-04-12T19:30:07+00:00"
    },
    {
      "id": "66a5345202e04f2b99f7e1cf979899ce",
      "sender": "achow101",
      "payload": "ok",
      "action": false,
      "timestamp": "2019-04-12T19:30:32+00:00"
    },
    {
      "id": "50fccf763fb14479bab35f15b1433727",
      "sender": "achow101",
      "payload": "jnewbery: the upgrade stuff I think will be inherently dangerous. shoehorning the old ismine logic into the new ismine logic is not trivial and they are incompatible in many different ways",
      "action": false,
      "timestamp": "2019-04-12T19:31:03+00:00"
    },
    {
      "id": "51ec7bdc5727465cb5e71de40fe88dca",
      "sender": "ryanofsky",
      "payload": "achow101, you don't even need to add upgrade in the initial pr",
      "action": false,
      "timestamp": "2019-04-12T19:31:35+00:00"
    },
    {
      "id": "e4e68a9345a147d8a4fe5f0474017ae9",
      "sender": "ryanofsky",
      "payload": "we can add support for just creating new descriptor wallets, or just importing descriptors first",
      "action": false,
      "timestamp": "2019-04-12T19:32:10+00:00"
    },
    {
      "id": "30d77ca8a11d479ea1d96326d708ab0f",
      "sender": "jnewbery",
      "payload": "i don't agree that it's inherently dangerous. We just need to do lots of testing until we're satisfied that it's no longer dangerous",
      "action": false,
      "timestamp": "2019-04-12T19:32:28+00:00"
    },
    {
      "id": "69a00d2889e849c195bf2ba0ae9036dc",
      "sender": "meshcollider",
      "payload": "provoostenator is having IRC issues and can't send messages here atm but would like to point out he also has a slightly different (and perhaps less complete) PR open: https://github.com/bitcoin/bitcoin/pull/15487",
      "action": false,
      "timestamp": "2019-04-12T19:32:30+00:00"
    },
    {
      "id": "59e8d4e3872c4de9ab3d50592805d5be",
      "sender": "jnewbery",
      "payload": "and roll it out slowly, with warnings to backup, etc",
      "action": false,
      "timestamp": "2019-04-12T19:32:45+00:00"
    },
    {
      "id": "bb65197c4ede4478838325d9ee1b307d",
      "sender": "achow101",
      "payload": "ryanofsky: right",
      "action": false,
      "timestamp": "2019-04-12T19:32:49+00:00"
    },
    {
      "id": "b23eb2e18e4248a2ae1d8cf3c4bb92eb",
      "sender": "jnewbery",
      "payload": "Everything ryanofsky is suggesting is still possible by using wallet versions and not flags",
      "action": false,
      "timestamp": "2019-04-12T19:33:36+00:00"
    },
    {
      "id": "2bb89635e3264ef2b4d72b746b95e716",
      "sender": "achow101",
      "payload": "ryanofsky: we shouldn't allow people to create mixed descriptor and non-descriptor wllets though. so no importing descriptors",
      "action": false,
      "timestamp": "2019-04-12T19:33:37+00:00"
    },
    {
      "id": "0c53ed668f1741c5bd917e822bdf316d",
      "sender": "achow101",
      "payload": "jnewbery: making it optional is not",
      "action": false,
      "timestamp": "2019-04-12T19:34:05+00:00"
    },
    {
      "id": "77abb96fd9b1442183c2778eee9a6106",
      "sender": "achow101",
      "payload": "that should only be done with flags",
      "action": false,
      "timestamp": "2019-04-12T19:34:19+00:00"
    },
    {
      "id": "f9f0591cfb85474da67e81b9b419791a",
      "sender": "jnewbery",
      "payload": "why not? Add a new parameter to createwallet",
      "action": false,
      "timestamp": "2019-04-12T19:34:29+00:00"
    },
    {
      "id": "ed9dbc6644504a2892ba3cd4ec3bc272",
      "sender": "ryanofsky",
      "payload": "yeah, i'm not sure you actually need a version or a flag",
      "action": false,
      "timestamp": "2019-04-12T19:34:50+00:00"
    },
    {
      "id": "163a8991598e4d3e9a9184b728eb1ac4",
      "sender": "jnewbery",
      "payload": "if descriptorwallet=false create an old-style wallet, if true create a descriptor wallet",
      "action": false,
      "timestamp": "2019-04-12T19:35:05+00:00"
    },
    {
      "id": "11f97f676418468e913e866fd02ce626",
      "sender": "achow101",
      "payload": "the last time we did optional version was hd wallet and that was a headache to reconcile with upgrades in the future. I would rather not go through that excercise again",
      "action": false,
      "timestamp": "2019-04-12T19:35:17+00:00"
    },
    {
      "id": "08c0628318f8468ca22fe10da123084b",
      "sender": "jnewbery",
      "payload": "default to false",
      "action": false,
      "timestamp": "2019-04-12T19:35:18+00:00"
    },
    {
      "id": "915e7598baa24f14b41ab7d6c708fbb4",
      "sender": "jnewbery",
      "payload": "we need to handle upgrades anyway. You already have code to do that",
      "action": false,
      "timestamp": "2019-04-12T19:35:37+00:00"
    },
    {
      "id": "1abc270828e1485485f9659d91586cb6",
      "sender": "provoostenator",
      "payload": "hi?",
      "action": false,
      "timestamp": "2019-04-12T19:36:05+00:00"
    },
    {
      "id": "d1cce5b4bf1f49e5a54cec9287c57da9",
      "sender": "harding",
      "payload": "provoostenator: hi",
      "action": false,
      "timestamp": "2019-04-12T19:36:09+00:00"
    },
    {
      "id": "045681d6fee84cf6895bfed985f55653",
      "sender": "provoostenator",
      "payload": "YES!",
      "action": false,
      "timestamp": "2019-04-12T19:36:12+00:00"
    },
    {
      "id": "640b18e8acbc4f6d8c943500e203a4a0",
      "sender": "meshcollider",
      "payload": "provoostenator: hi \\o/",
      "action": false,
      "timestamp": "2019-04-12T19:36:18+00:00"
    },
    {
      "id": "2a73083a66224c619701adc53c26c738",
      "sender": "jnewbery",
      "payload": "hi sjors!",
      "action": false,
      "timestamp": "2019-04-12T19:36:23+00:00"
    },
    {
      "id": "539a452d78a348fc995f6ecdb2c93c0d",
      "sender": "provoostenator",
      "payload": "That was weird, I've been talking into a void for a day or so :-)",
      "action": false,
      "timestamp": "2019-04-12T19:36:24+00:00"
    },
    {
      "id": "474ea06149f940a1a34af1d46a88bb86",
      "sender": "provoostenator",
      "payload": "In my version of descriptor wallets it's a feature flag and opt-in. Obviously this sort of thing is easy to tweak.",
      "action": false,
      "timestamp": "2019-04-12T19:37:03+00:00"
    },
    {
      "id": "05ff6656386443ac8b070870a81647e1",
      "sender": "achow101",
      "payload": "jnewbery: I think we should maintain the separation of wallet flags for optional, wallet version for mandatory",
      "action": false,
      "timestamp": "2019-04-12T19:37:16+00:00"
    },
    {
      "id": "606a12a7f1194035985cdd7022afefbc",
      "sender": "provoostenator",
      "payload": "I'll do a more thorough comparison later.",
      "action": false,
      "timestamp": "2019-04-12T19:37:17+00:00"
    },
    {
      "id": "538d75093ca54fce84e505a9047ee455",
      "sender": "jnewbery",
      "payload": "wallet version is optional. Upgrading from an old version to new is optional",
      "action": false,
      "timestamp": "2019-04-12T19:37:58+00:00"
    },
    {
      "id": "0d5b3ccdbd9348a6978ceecbc3a42aae",
      "sender": "meshcollider",
      "payload": "achow101: but why not just allow the wallet to be created with either version number",
      "action": false,
      "timestamp": "2019-04-12T19:37:59+00:00"
    },
    {
      "id": "af7e03fedd394c178ed85607864d2c4c",
      "sender": "jnewbery",
      "payload": "right",
      "action": false,
      "timestamp": "2019-04-12T19:38:06+00:00"
    },
    {
      "id": "34c3d4cf73554bbc98c75e96881a470c",
      "sender": "ryanofsky",
      "payload": "jnewbery, i think version numbers are just confusing and should never be used again",
      "action": false,
      "timestamp": "2019-04-12T19:38:14+00:00"
    },
    {
      "id": "f63c5c7e5a3d48459b8795369cbe1181",
      "sender": "jnewbery",
      "payload": "They are confusing, but they at least cut down on the number of combinations of options",
      "action": false,
      "timestamp": "2019-04-12T19:38:52+00:00"
    },
    {
      "id": "7196251c4eec46fb931886c8bcf1ae78",
      "sender": "ryanofsky",
      "payload": "your concern about testing and flag combinations is easily addressed by just refusing to load / support wallets with whatever combinations of flags you want to rule out",
      "action": false,
      "timestamp": "2019-04-12T19:38:52+00:00"
    },
    {
      "id": "edd2eb4100114b8682e466d1ac4011f4",
      "sender": "jnewbery",
      "payload": "I think that's effectively the same thing, no?",
      "action": false,
      "timestamp": "2019-04-12T19:39:26+00:00"
    },
    {
      "id": "52fea5c5a8c2429d8b15c1fc81cc6c5d",
      "sender": "meshcollider",
      "payload": "IIRC in lightning, when exchanging feature lists, there are some optional features and some mandatory features based on bit positions",
      "action": false,
      "timestamp": "2019-04-12T19:39:33+00:00"
    },
    {
      "id": "1f7f36de6fca43c58d86fc758799d6b2",
      "sender": "ryanofsky",
      "payload": "yes, exactly",
      "action": false,
      "timestamp": "2019-04-12T19:39:36+00:00"
    },
    {
      "id": "081a87c1424b4a979fd26e8b0fd48380",
      "sender": "achow101",
      "payload": "meshcollider: jnewbery: the problem with optional wallet versions is that any future wallet version becomes optional as well. You can't e.g. skip descriptor wallets and go to the version after that which introduces something completely unrelated to descriptors",
      "action": false,
      "timestamp": "2019-04-12T19:39:49+00:00"
    },
    {
      "id": "575b651ff3c24063832a2936e15f71c3",
      "sender": "ryanofsky",
      "payload": "anything you want to do with version numbers is possible with flags, but flags are more readable and easier to think about",
      "action": false,
      "timestamp": "2019-04-12T19:39:56+00:00"
    },
    {
      "id": "205f680165ac49f3b56a5e2b8ad863f1",
      "sender": "achow101",
      "payload": "reconciling the two becomes a pain, as it was for hd wallets",
      "action": false,
      "timestamp": "2019-04-12T19:40:00+00:00"
    },
    {
      "id": "2b59d8a4f28e4c2f98130265cbc2af18",
      "sender": "jnewbery",
      "payload": "achow101 that's always the case with version numbers",
      "action": false,
      "timestamp": "2019-04-12T19:40:17+00:00"
    },
    {
      "id": "90a72f697174492180ceb2acd50272ea",
      "sender": "meshcollider",
      "payload": "We dont have to add any new features to old style wallets from now on",
      "action": false,
      "timestamp": "2019-04-12T19:40:24+00:00"
    },
    {
      "id": "cedb5ac665f0464c889456a45f5d37cf",
      "sender": "meshcollider",
      "payload": "Force a descriptor upgrade before any other upgrade",
      "action": false,
      "timestamp": "2019-04-12T19:40:35+00:00"
    },
    {
      "id": "6f09ad41818c4c29991f1a6376d27d5d",
      "sender": "ryanofsky",
      "payload": "because you don't have to mentally load the whole project history to figure out what conditions are possible, it's just written explicitly in code",
      "action": false,
      "timestamp": "2019-04-12T19:40:45+00:00"
    },
    {
      "id": "3e0c8baadd7d4ba08c9fa8d18c311000",
      "sender": "provoostenator",
      "payload": "I'd rather not force upgrades anytime soon. That most likely will lead to endless delays in shipping.",
      "action": false,
      "timestamp": "2019-04-12T19:41:08+00:00"
    },
    {
      "id": "39abf5b1d10043d59c6b336be34c765a",
      "sender": "jnewbery",
      "payload": "even if you don't make the version 'optional', there are still users of old wallets who would have to upgrade through descriptor wallets if they wanted a later version",
      "action": false,
      "timestamp": "2019-04-12T19:41:11+00:00"
    },
    {
      "id": "da05751dbeb24b74bd55f7fd0322b1a6",
      "sender": "jnewbery",
      "payload": "provoostenator: definitely. No forced upgrades",
      "action": false,
      "timestamp": "2019-04-12T19:41:34+00:00"
    },
    {
      "id": "7d9b89e48cf74d8ca0ed683559a84ba0",
      "sender": "achow101",
      "payload": "i agree with ryanofsky",
      "action": false,
      "timestamp": "2019-04-12T19:42:12+00:00"
    },
    {
      "id": "80a3e4a2920342bc88e6ae2fb29b97d5",
      "sender": "achow101",
      "payload": "anyways, this is bikeshedding",
      "action": false,
      "timestamp": "2019-04-12T19:42:25+00:00"
    },
    {
      "id": "02aedfda1a164c5ba5ce8e9f3b50464a",
      "sender": "jnewbery",
      "payload": "?",
      "action": false,
      "timestamp": "2019-04-12T19:42:32+00:00"
    },
    {
      "id": "d8b1ae7a92984a4da64e914da111735b",
      "sender": "meshcollider",
      "payload": "I think this is important for the approach",
      "action": false,
      "timestamp": "2019-04-12T19:42:51+00:00"
    },
    {
      "id": "9c88a990ecf4477d820afe69e1142e9e",
      "sender": "provoostenator",
      "payload": "What's more important I think is to decide if we want to support a hybrid with descriptors and regular stuff (I prefer not).",
      "action": false,
      "timestamp": "2019-04-12T19:43:20+00:00"
    },
    {
      "id": "eef82e576fba4479aa2a3e4be1ad079d",
      "sender": "achow101",
      "payload": "provoostenator: definitely no",
      "action": false,
      "timestamp": "2019-04-12T19:43:34+00:00"
    },
    {
      "id": "83a069da9ba442d38a07ac92484a5d97",
      "sender": "meshcollider",
      "payload": "No, we shouldnt",
      "action": false,
      "timestamp": "2019-04-12T19:43:38+00:00"
    },
    {
      "id": "cc2652c856264b62a469ced4db496eac",
      "sender": "provoostenator",
      "payload": "Ok, so that can be supported both with versioning and with feature flags I think. We can revisit that later?",
      "action": false,
      "timestamp": "2019-04-12T19:44:54+00:00"
    },
    {
      "id": "a1dd6d5e645c4b24a171deb0f94a5f32",
      "sender": "provoostenator",
      "payload": "Or is there something where this choice does matter more urgently?",
      "action": false,
      "timestamp": "2019-04-12T19:45:17+00:00"
    },
    {
      "id": "eb8360170c8a4f59ab7b97b2a07609ae",
      "sender": "achow101",
      "payload": "meshcollider: jnewbery: consider the case where we introduce descriptor wallets as a new optional version. Sometime later down the road, we introduce a new wallet version because a new field is introduced in the wallet that is inherently not backwards compatible so it needs a version bump to prevent old software from loading, e.g. several more wallet flags",
      "action": false,
      "timestamp": "2019-04-12T19:45:29+00:00"
    },
    {
      "id": "07f89ef24f364a0ba128931737a57055",
      "sender": "meshcollider",
      "payload": "I think it just affects how to deal with transitioning tests",
      "action": false,
      "timestamp": "2019-04-12T19:45:44+00:00"
    },
    {
      "id": "456e259a8e824879b433f04d07565f5e",
      "sender": "achow101",
      "payload": "in order for users to upgrade to that version with more wallet flags, they first have to upgrade to descriptors, which they may not want to. that new version provides new functionality that does not require descriptors but is still applicable to old wallets. now this needs to be reconciled, and that's rather difficult",
      "action": false,
      "timestamp": "2019-04-12T19:46:24+00:00"
    },
    {
      "id": "e25efa0619cb4ca788f4d4fa3e8fe297",
      "sender": "ryanofsky",
      "payload": "yeah, i don't see why flags aren't just obviously better in every case. you can still prevent combinations of features, you are just forced to write down which combinations are possible",
      "action": false,
      "timestamp": "2019-04-12T19:46:37+00:00"
    },
    {
      "id": "ac2031acbc31486ab707020c69e0fb14",
      "sender": "jnewbery",
      "payload": "achow101 are you arguing that this should be a flag?",
      "action": false,
      "timestamp": "2019-04-12T19:46:48+00:00"
    },
    {
      "id": "38eabe008cb645be846b8ee8c0a5cead",
      "sender": "meshcollider",
      "payload": "Flags alone dont prevent old nodes from trying to open though do they?",
      "action": false,
      "timestamp": "2019-04-12T19:47:04+00:00"
    },
    {
      "id": "8842991c61204116b4fb5a99e8b030ad",
      "sender": "achow101",
      "payload": "jnewbery: yes. if it is going to be an optional feature, it should be a flag",
      "action": false,
      "timestamp": "2019-04-12T19:47:08+00:00"
    },
    {
      "id": "cc8d5ecf977e4899a026fc62c432c8de",
      "sender": "ryanofsky",
      "payload": "meshcollider, mandatory flags do",
      "action": false,
      "timestamp": "2019-04-12T19:47:23+00:00"
    },
    {
      "id": "c2aa6f4ce80f43eca5cd93ecce006a1d",
      "sender": "jnewbery",
      "payload": "Of course it's an optional feature. We don't force people to upgrade!",
      "action": false,
      "timestamp": "2019-04-12T19:47:49+00:00"
    },
    {
      "id": "58f2078200484c4ca737eced99cee03d",
      "sender": "meshcollider",
      "payload": "Ok yes I agree with making it a mandatory flag rather than version",
      "action": false,
      "timestamp": "2019-04-12T19:48:02+00:00"
    },
    {
      "id": "85510c240fff4d92b41b6702d73d0da0",
      "sender": "jnewbery",
      "payload": "I'm fine with it being a flag. Like ryanofsky has said, you can do everything with flags that you can with versions (and more)",
      "action": false,
      "timestamp": "2019-04-12T19:48:28+00:00"
    },
    {
      "id": "5dc7bf5cfb4945d2bb15cf1cd46e3479",
      "sender": "jnewbery",
      "payload": "As long as the valid combinations are all documented and enforced in one place",
      "action": false,
      "timestamp": "2019-04-12T19:49:14+00:00"
    },
    {
      "id": "d8860dff30d3454fbfd6af4a7080a8bb",
      "sender": "provoostenator",
      "payload": "A mandatory flag also means we can reduce complexity.",
      "action": false,
      "timestamp": "2019-04-12T19:49:17+00:00"
    },
    {
      "id": "facf23771bde427f86b9736069063963",
      "sender": "provoostenator",
      "payload": "Only supporting upgrades for basic cases.",
      "action": false,
      "timestamp": "2019-04-12T19:49:40+00:00"
    },
    {
      "id": "cdb75b506cc04b97832fa8d262547dbf",
      "sender": "achow101",
      "payload": "alright, i'll change the pr to use a flag, restore the tests, and drop the upgrading logic for now",
      "action": false,
      "timestamp": "2019-04-12T19:49:51+00:00"
    },
    {
      "id": "760605d989514b009eec382890bdb7c1",
      "sender": "meshcollider",
      "payload": "Just because its a flag doesn't mean we have to allow all possible combinations of flags",
      "action": false,
      "timestamp": "2019-04-12T19:50:02+00:00"
    },
    {
      "id": "e2cfe216779d4df880b07c76262e7f2a",
      "sender": "meshcollider",
      "payload": "achow101: +1",
      "action": false,
      "timestamp": "2019-04-12T19:50:43+00:00"
    },
    {
      "id": "3dc879c6914f41f5b167854014d44253",
      "sender": "meshcollider",
      "payload": "Anything else for the last 10 minutes?",
      "action": false,
      "timestamp": "2019-04-12T19:50:56+00:00"
    },
    {
      "id": "20f5dfced15e47d698dc5f95c783ca61",
      "sender": "meshcollider",
      "payload": "Anything else for backport to 0.18.0rc4 or high priority requests?",
      "action": false,
      "timestamp": "2019-04-12T19:51:22+00:00"
    },
    {
      "id": "c5055d166c5d40b7a01b0ac8cd060fac",
      "sender": "jnewbery",
      "payload": "Wallet high priority PRs are #15006 #14447 #15741",
      "action": false,
      "timestamp": "2019-04-12T19:51:50+00:00"
    },
    {
      "id": "d68b17d78c8345b6b8f80fab8efebc6f",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/15006 | Add option to create an encrypted wallet by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #15006 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2019-04-12T19:51:54+00:00"
    },
    {
      "id": "43e5361060a74cc5ba3a4a5ca491c266",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/14447 | Armory 0.96.4 causies BitcoinCore 0.17 crash sometimes \u00c3\u0082\u00c2\u00b7 Issue #14447 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2019-04-12T19:51:55+00:00"
    },
    {
      "id": "9f873bddf04a47bbba46b2b9e9cf69e6",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/15741 | Batch write imported stuff in importmulti by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #15741 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2019-04-12T19:51:57+00:00"
    },
    {
      "id": "fd4cc147c7214c12950961d420f6413e",
      "sender": "jnewbery",
      "payload": "sorry #15557",
      "action": false,
      "timestamp": "2019-04-12T19:52:02+00:00"
    },
    {
      "id": "297e5d1cb8db4f06ae8260aabcb9ccf8",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/15557 | Enhance `bumpfee` to include inputs when targeting a feerate by instagibbs \u00c3\u0082\u00c2\u00b7 Pull Request #15557 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2019-04-12T19:52:06+00:00"
    },
    {
      "id": "d6cb0b74455142adbfbc1678c41f37ed",
      "sender": "jnewbery",
      "payload": "I think 15557 is almost ready for merge. It's a really nice feature and quite easy to review",
      "action": false,
      "timestamp": "2019-04-12T19:52:33+00:00"
    },
    {
      "id": "656916a3f3bd41408ab1fa3bf0eea2a9",
      "sender": "meshcollider",
      "payload": "Maybe kallewoof would like to have that debit/credit caching one on there",
      "action": false,
      "timestamp": "2019-04-12T19:53:06+00:00"
    },
    {
      "id": "932cd78269cb46a5be33729a523b582b",
      "sender": "meshcollider",
      "payload": "#15780",
      "action": false,
      "timestamp": "2019-04-12T19:53:17+00:00"
    },
    {
      "id": "cbd577f3a24445e984c33ad1d5cd8aa1",
      "sender": "gribble",
      "payload": "https://github.com/bitcoin/bitcoin/issues/15780 | wallet: add cachable accounts for caching credit/debit values by kallewoof \u00c3\u0082\u00c2\u00b7 Pull Request #15780 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
      "action": false,
      "timestamp": "2019-04-12T19:53:20+00:00"
    },
    {
      "id": "efcc3b7971de46cbb4b0e517e2f99101",
      "sender": "jnewbery",
      "payload": "I think that can go on. It blocks his other PR",
      "action": false,
      "timestamp": "2019-04-12T19:54:04+00:00"
    },
    {
      "id": "7ff0b93261664ef5ba6b7957c3d9300c",
      "sender": "jnewbery",
      "payload": "(which has been open for 8 months)",
      "action": false,
      "timestamp": "2019-04-12T19:54:26+00:00"
    },
    {
      "id": "75cb0412db6643dea483b23d55c39adb",
      "sender": "meshcollider",
      "payload": "Yes, and he is very proactive with rebasing, he deserves some acceleration lol",
      "action": false,
      "timestamp": "2019-04-12T19:55:12+00:00"
    },
    {
      "id": "1964900375dd4d6b81a38fad385d5b65",
      "sender": "jnewbery",
      "payload": "#action review more kallewoof PRs!",
      "action": false,
      "timestamp": "2019-04-12T19:55:30+00:00"
    },
    {
      "id": "416b47b183f94adebc42c66961d81bdb",
      "sender": "meshcollider",
      "payload": "Ok seems thats all for the meeting today then :)",
      "action": false,
      "timestamp": "2019-04-12T19:56:11+00:00"
    },
    {
      "id": "043291c50e9845a1a40a9d232988fda1",
      "sender": "meshcollider",
      "payload": "#endmeeting",
      "action": false,
      "timestamp": "2019-04-12T19:56:16+00:00"
    }
  ],
  "events": [
    {
      "event_type": "START_MEETING",
      "message": {
        "id": "62e539ee63444e829a2e73d6f6b4b4e8",
        "sender": "meshcollider",
        "payload": "#startmeeting",
        "action": false,
        "timestamp": "2019-04-12T19:00:04+00:00"
      },
      "operand": null,
      "id": "62e539ee63444e829a2e73d6f6b4b4e8",
      "timestamp": "2019-04-12T19:00:04+00:00"
    },
    {
      "event_type": "TOPIC",
      "message": {
        "id": "6584f639ad9e46bda5ecebdad4dc46b9",
        "sender": "meshcollider",
        "payload": "#topic : Upgrade wallet (#15761) (jnewbery)",
        "action": false,
        "timestamp": "2019-04-12T19:01:35+00:00"
      },
      "operand": ": Upgrade wallet (#15761) (jnewbery)",
      "id": "6584f639ad9e46bda5ecebdad4dc46b9",
      "timestamp": "2019-04-12T19:01:35+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "cdb8a1efd95249d9a03ba15451178da3",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/15761 | Replace -upgradewallet startup option with upgradewallet RPC by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #15761 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2019-04-12T19:01:37+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/15761",
      "id": "cdb8a1efd95249d9a03ba15451178da3",
      "timestamp": "2019-04-12T19:01:37+00:00"
    },
    {
      "event_type": "TOPIC",
      "message": {
        "id": "a579447284584824bc35309ea5eeb06b",
        "sender": "meshcollider",
        "payload": "#topic native descriptor wallets (achow101)",
        "action": false,
        "timestamp": "2019-04-12T19:17:31+00:00"
      },
      "operand": "native descriptor wallets (achow101)",
      "id": "a579447284584824bc35309ea5eeb06b",
      "timestamp": "2019-04-12T19:17:31+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "35a4b37d9051458984d96c310718e44d",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/15764 | Native descriptor wallets by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #15764 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2019-04-12T19:18:03+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/15764",
      "id": "35a4b37d9051458984d96c310718e44d",
      "timestamp": "2019-04-12T19:18:03+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "d68b17d78c8345b6b8f80fab8efebc6f",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/15006 | Add option to create an encrypted wallet by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #15006 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2019-04-12T19:51:54+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/15006",
      "id": "d68b17d78c8345b6b8f80fab8efebc6f",
      "timestamp": "2019-04-12T19:51:54+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "43e5361060a74cc5ba3a4a5ca491c266",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/14447 | Armory 0.96.4 causies BitcoinCore 0.17 crash sometimes \u00c3\u0082\u00c2\u00b7 Issue #14447 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2019-04-12T19:51:55+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/14447",
      "id": "43e5361060a74cc5ba3a4a5ca491c266",
      "timestamp": "2019-04-12T19:51:55+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "9f873bddf04a47bbba46b2b9e9cf69e6",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/15741 | Batch write imported stuff in importmulti by achow101 \u00c3\u0082\u00c2\u00b7 Pull Request #15741 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2019-04-12T19:51:57+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/15741",
      "id": "9f873bddf04a47bbba46b2b9e9cf69e6",
      "timestamp": "2019-04-12T19:51:57+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "297e5d1cb8db4f06ae8260aabcb9ccf8",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/15557 | Enhance `bumpfee` to include inputs when targeting a feerate by instagibbs \u00c3\u0082\u00c2\u00b7 Pull Request #15557 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2019-04-12T19:52:06+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/15557",
      "id": "297e5d1cb8db4f06ae8260aabcb9ccf8",
      "timestamp": "2019-04-12T19:52:06+00:00"
    },
    {
      "event_type": "LINK",
      "message": {
        "id": "cbd577f3a24445e984c33ad1d5cd8aa1",
        "sender": "gribble",
        "payload": "https://github.com/bitcoin/bitcoin/issues/15780 | wallet: add cachable accounts for caching credit/debit values by kallewoof \u00c3\u0082\u00c2\u00b7 Pull Request #15780 \u00c3\u0082\u00c2\u00b7 bitcoin/bitcoin \u00c3\u0082\u00c2\u00b7 GitHub",
        "action": false,
        "timestamp": "2019-04-12T19:53:20+00:00"
      },
      "operand": "https://github.com/bitcoin/bitcoin/issues/15780",
      "id": "cbd577f3a24445e984c33ad1d5cd8aa1",
      "timestamp": "2019-04-12T19:53:20+00:00"
    },
    {
      "event_type": "ACTION",
      "message": {
        "id": "1964900375dd4d6b81a38fad385d5b65",
        "sender": "jnewbery",
        "payload": "#action review more kallewoof PRs!",
        "action": false,
        "timestamp": "2019-04-12T19:55:30+00:00"
      },
      "operand": "review more kallewoof PRs!",
      "id": "1964900375dd4d6b81a38fad385d5b65",
      "timestamp": "2019-04-12T19:55:30+00:00"
    },
    {
      "event_type": "END_MEETING",
      "message": {
        "id": "043291c50e9845a1a40a9d232988fda1",
        "sender": "meshcollider",
        "payload": "#endmeeting",
        "action": false,
        "timestamp": "2019-04-12T19:56:16+00:00"
      },
      "operand": null,
      "id": "043291c50e9845a1a40a9d232988fda1",
      "timestamp": "2019-04-12T19:56:16+00:00"
    }
  ],
  "aliases": {},
  "vote_in_progress": false,
  "motion_index": null
}