Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Move all possible events to serverside #1501

Closed
arikfr opened this issue Jan 2, 2017 · 3 comments
Closed

Move all possible events to serverside #1501

arikfr opened this issue Jan 2, 2017 · 3 comments
Labels
Tech Debt

Comments

@arikfr
Copy link
Member

arikfr commented Jan 2, 2017

Currently we log some events from the client and some events from the server. We should log all possible events from the server and leave on the client only those that we can't.
@alison985 alison985 mentioned this issue Sep 1, 2017
Closed
@alison985
Copy link
Contributor

@arikfr I did 2 examples of this in case to start out with in case you want to review: alison985@5d8b208

@alison985
Copy link
Contributor

For all of the ones I could find a way to do server-side: mozilla#246

@arikfr
Copy link
Member Author

arikfr commented Sep 6, 2017

Thanks! Commented on the commit.

@jezdez jezdez mentioned this issue Jun 19, 2018
Closed
alison985 added a commit to StantonVentures/redash that referenced this issue Jul 1, 2018
@alison985
move events serverside
f31e346 
addresses getredash#1501
@alison985 alison985 mentioned this issue Jul 1, 2018
Closed
alison985 added a commit to StantonVentures/redash that referenced this issue Jul 22, 2018
@alison985
move events serverside
4ac7ab8 
addresses getredash#1501
alison985 added a commit to StantonVentures/redash that referenced this issue Aug 20, 2018
@alison985
move events serverside
5497f0e 
addresses getredash#1501
alison985 added a commit to StantonVentures/redash that referenced this issue Aug 20, 2018
@alison985
move events serverside
ea4d2ab 
addresses getredash#1501
@arikfr arikfr closed this as completed Dec 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Tech Debt
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@arikfr @alison985