Skip to content
This repository has been archived by the owner on Mar 22, 2019. It is now read-only.

[WIP] #198 Online Status #244

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

[WIP] #198 Online Status #244

wants to merge 7 commits into from

Conversation

appinteractive
Copy link
Member

@appinteractive appinteractive commented Mar 14, 2019

Save the current lastActiveAt timestamp on the user when requesting the api.

Todos

  • Write tests
  • Let user decide if last date is shown to other users
  • look into queuing the timestamp updating

Related with

Human-Connection/Nitro-Web#262

Copy link
Contributor

@roschaefer roschaefer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@appinteractive do we really store this in Neo4J? Apollo Server is good at fetching data and/or writing to various services.

This feature would be one of the exceptions where I would be interested in the performance impact.

To me this is more a logging functionality. How about researching a good logging solution before rolling our own?

@roschaefer
Copy link
Contributor

To me it would also be interesting to see not only the last time active, but a visualization of activity. Maybe there is a good solution out there already?

@appinteractive
Copy link
Member Author

appinteractive commented Mar 14, 2019

What do you mean? It’s just the timestamp of the last activity to be able to show if the user was active in the last 15 minutes, the same that is done in the current alpha. It has nothing to do with logging or audit.

But I agree that we could throttle it with a small queue for that for saving every 30 sek or something like that.

This was just the first shot to validate the idea.

And what did you mean with visualization?

@appinteractive
Copy link
Member Author

appinteractive commented Mar 15, 2019

@roschaefer now its queued and updated in batch! should be enough for the moment.

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

Successfully merging this pull request may close these issues.

3 participants