{ "type": "pull-request", "repo": "%MeCTQrz9uszf9EZoTnKCeFeIedhnKWuB3JHW2l1g9NA=.sha256", "project": "%MeCTQrz9uszf9EZoTnKCeFeIedhnKWuB3JHW2l1g9NA=.sha256", "branch": "master", "head_repo": "%MeCTQrz9uszf9EZoTnKCeFeIedhnKWuB3JHW2l1g9NA=.sha256", "head_branch": "opt-in", "text": "## Implement opt-in for publicWebHosting\n\nExpressed interest:\n\n[@nanomonkey](@+D0ku/LReK6kqd3PSrcVCfbLYbDtTmS4Bd21rqhpYNA=.ed25519) %xcPZC7P5Pl2IIAVggFGRe8hU/1NMKsW9Gx2gBO/vQto=.sha256\n[@zero](@+D0ku/LReK6kqd3PSrcVCfbLYbDtTmS4Bd21rqhpYNA=.ed25519) %fyVm+jWmvq9JIR4YY2ekt6zh2EVQomimQ3gqCzPcieU=.sha256\n[@staltzphone](@+UMKhpbzXAII+2/7ZlsgkJwIsxdfeFi36Z5Rk1gCfY0=.ed25519) %7aEuviRwcHxO7hyRvSNu+4AptBzavNzOwP20CYavKvU=.sha256\n[@kas](@dBQlwh9Gtr3i5YMOGtIOKtGNVepeu+nyb6KGl1vtOcM=.ed25519) %tQbb6TeQLLeoET9sHksAUPEjmkyDbTaT5jmWnbKfkaI=.sha256\n\nThis treats feeds that have not set their publicWebHosting setting to be as if they set it to false - except it will say they have \"not chosen to be\" hosted publicly instead of \"chosen not to be\".\n\nAlso I changed it to respect feeds setting publicWebHosting:null, in case anyone wants to clear their preference and let the viewer use its default setting.\n\nAnd names of hidden feeds will now be hidden also.\n\nThe default can be changed back to the previous behavior (opt-out) by setting a config option, e.g. for use on local/private networks.\n\ncc: [@arj](@6CAxOI3f+LUOVrbAl0IemqiS7ATpQvr9Mdw9LC4+Uv0=.ed25519)", "mentions": [ { "link": "@+D0ku/LReK6kqd3PSrcVCfbLYbDtTmS4Bd21rqhpYNA=.ed25519", "name": "nanomonkey" }, { "link": "%xcPZC7P5Pl2IIAVggFGRe8hU/1NMKsW9Gx2gBO/vQto=.sha256" }, { "link": "@+D0ku/LReK6kqd3PSrcVCfbLYbDtTmS4Bd21rqhpYNA=.ed25519", "name": "zero" }, { "link": "%fyVm+jWmvq9JIR4YY2ekt6zh2EVQomimQ3gqCzPcieU=.sha256" }, { "link": "@+UMKhpbzXAII+2/7ZlsgkJwIsxdfeFi36Z5Rk1gCfY0=.ed25519", "name": "staltzphone" }, { "link": "%7aEuviRwcHxO7hyRvSNu+4AptBzavNzOwP20CYavKvU=.sha256" }, { "link": "@dBQlwh9Gtr3i5YMOGtIOKtGNVepeu+nyb6KGl1vtOcM=.ed25519", "name": "kas" }, { "link": "%tQbb6TeQLLeoET9sHksAUPEjmkyDbTaT5jmWnbKfkaI=.sha256" }, { "link": "@6CAxOI3f+LUOVrbAl0IemqiS7ATpQvr9Mdw9LC4+Uv0=.ed25519", "name": "arj" } ] }
Looks good cel!
I'll deploy this to my pub when you merge.
Thanks!
Although it would probably be good to have client support for this when we release it. The biggest problem with the old way, was that people had to use a command line to disable it.
Did a git update
- Fix requireOptIn option
@arj I'm going ahead and deploying this even without client support, since I feel the call for this change has been great
Deployed to between-two-worlds.dk as well. I'll see about adding support for this to #patchbay