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

Create events_log table #287

Open
jaredatron opened this issue Sep 20, 2017 · 3 comments
Open

Create events_log table #287

jaredatron opened this issue Sep 20, 2017 · 3 comments
Assignees

Comments

@jaredatron
Copy link
Contributor

jaredatron commented Sep 20, 2017

Right now we store when a learner checks a skill checkbox in the table checks. We also log every time a learner checks or unchecks a skill checkbox in check_log.

We want to log more than just the checks now that were about to add support for checking challenge checkboxes.

  • Create the event_log table with the following columns:
    • user_id (integer)
    • occurred_at (Datetime)
    • metadata (PG Hash)
  • Migrate the check_log table into another table called event_logs with a new column of type which is a string.
  • All rows migrated from event_logs should have the type skill_check.
  • label, checked and referrer should be moved into properties of the metadata column hash
  • Update the queries in /database to retrieve the log of skill checks from the new table.
@jaredatron jaredatron changed the title migrate check_log to events_log table and start logging "all" events Create events_log table and start logging "all" events Sep 20, 2017
@jaredatron jaredatron changed the title Create events_log table and start logging "all" events Create events_log table Sep 20, 2017
@je-lopez
Copy link
Contributor

@je-lopez
Copy link
Contributor

@je-lopez
Copy link
Contributor

latest pr with your refactoring: https://github.com/LearnersGuild/curriculum/pull/333

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

No branches or pull requests

3 participants