Skip to content
This repository has been archived by the owner on Jan 23, 2018. It is now read-only.

The scrollback is not quite right for the current population #92

Open
alexchally opened this issue Mar 7, 2015 · 4 comments
Open

The scrollback is not quite right for the current population #92

alexchally opened this issue Mar 7, 2015 · 4 comments

Comments

@alexchally
Copy link

I think that we need to revise how scrollback works a bit so that people don't log into an empty channel quite so often.

Maybe just dropping the time limit so it is always the last 30 messages, or maybe increasing the time limit to an hour instead of 10 minutes. There may be other ways of dealing with the situation, and I would like to open up a discussion about it so that we can evaluate the options as a community.

@cloudEdwards
Copy link

I agree alex, I was going to bring up this issue myself. There seems the be a perception that it will be full of messages saying "where's everyone?" But I don't think so. I think if, when you log in, you see other meats, you'll just say hi and try to interact. I think we could try it out, adn see how it works. We could always put it back the way it was if it doesn't work.

@cloudEdwards
Copy link

I am trying to find where the code is that decides how long the scroll back is. Travis was saying it isn't in the meatspace-chat-v2 repo. But in a module it depends on. I was looking at node-meatspace-publico repo. It might be in there in the .js somewhere. Any ideas as to where to find it? I think I found the part that limits the posts to the last 30.
node-meatspace-publica/main.js line 25 >>> this.limit = options.limit || 30;
Anyone got ideas?

edit ~ Jory is on the scent. He found the spot. It is in node-meatspace-publica/lib/services.js line 73

@MylesBorins
Copy link
Member

Potential solution... talk more?

@jasonrhodes
Copy link
Contributor

@thealphanerd +1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants