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.

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

Process Slack users.json #4

Open
louh opened this issue Nov 27, 2017 · 0 comments
Open

Process Slack users.json #4

louh opened this issue Nov 27, 2017 · 0 comments

Comments

@louh
Copy link
Member

louh commented Nov 27, 2017

via #3:

Process users.json:

  • Each user is an object in an array. This means looking up any user needs to iterate over the array and check the id property; this could get expensive. If we rewrite the data structure so that the id is a key lookup, matching users to the transcript gets easier to do.
  • Surface the useful information: first name and last name is the most important right now. Slack does not seem to use the same properties for everyone (it depends on what data the user provided) so we need to do some normalization.
  • Not all information is relevant and it may be dropped.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant