#[cleverdevil]I'm happy to work on that ben_thatmustbeme. I'm gonna need it for /together anyway.
#[cleverdevil]Maybe make a ticket in GitHub for me and mention? It'd be a fun project over vacation for me!
j12t, [pfefferle] and mapkyca joined the channel
#mapkyca!tell raucao: Just catching up - feeds on comments is already filed as an issue/feature request, but nobody has got to it yet. As others have also mentioned, there is a notifications page, but the link was removed in the last release because it needs some work ... if folk want to take that on that'd be awesome, I've certainly not had time :)
#raucaohi mapkyca. no need to have loqi tell me, i have a bouncer ;)
#Loqiraucao: mapkyca left you a message 1 minute ago: Just catching up - feeds on comments is already filed as an issue/feature request, but nobody has got to it yet. As others have also mentioned, there is a notifications page, but the link was removed in the last release because it needs some work ... if folk want to take that on that'd be awesome, I've certainly not had time :)
j12t joined the channel
#raucaohaven't done php in like 15 years, so i don't even know how it works anymore :)
#mapkycaNotifications has been repeatedly asked for, and a feed on comments would also be useful. I'll see if I can get to it at some point...
j12t, davidmead and [pfefferle] joined the channel
#mapkycaraucao: Put together a little pull request which brings back notifications, and among other things, adds an RSS feed to notification items :) ... will hold off on merging until I confer with ben and kyle, since notifications were removed for "UI reasons", but I'm not sure what those were, other than the read/unread class names being wrong. Meanwhile, feel free to kick the tires :)
#Loqimapkyca has 6 karma in this channel (13 overall)
j12t, tantek, [jeremycherfas], [miklb], [chrisaldrich] and [pfefferle] joined the channel
#mapkycaFyi, I've restored the notifications page for now, with some basic cosmetic work to make it a little more usable. Still needs a lot of work, and I think Ben and Erin want to explore different ways of representing this data, but since we're mid release it seems sensible to have some existing code to work with...
#[cleverdevil]But, notifications are a bit different than standard streams.
#tanteke.g. anything where otherwise you might say "just look at it in a reader" implies that a theme could have such a "reader stream" template in the theme that could be applied to any new h-feed page
#[cleverdevil]Well, I suppose it isn't *that* different.
#tantekcan you point to screenshots of examples that show such differences?
#[cleverdevil]Actually, now that I think about it, its really not that different...
#tantek[cleverdevil]: the point is, if someone could say "I just want to see it in a reader" - implies that they don't care for its presentation to be any different than a generic stream
#[cleverdevil]I'd have to look in more detail, but I definitely think that the current notifications page in Known could potentially be re-envisioned as an alternate "stream" type view.
#tantekthis is why I think any such "theme" should include a generic "stream page" template
#tantekso any new feature that shows more than one "item" could default to using that template
#tantekso it at least would look "ok" / good enough to ship
#tantekthen later, as a separate incremental step, if someone feels like it deserves more custom treatment, they could create a custom template for it. but keeping that as a second strictly optional step would help with more rapidly deploying new stream-like features.
travis-ci joined the channel
#travis-cimapkyca/idno#221 (master - 24bf8d5 : Marcus Povey): The build passed.