Skip to content

Conversation

@RobStallion
Copy link
Member

@RobStallion RobStallion commented Apr 25, 2019

@nelsonic following our convo I created an insert function that is not a macro to compare to macro alternative

@codecov
Copy link

codecov bot commented Apr 25, 2019

Codecov Report

❌ Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (master@9104b2d). Learn more about missing BASE report.

Files with missing lines Patch % Lines
lib/append/aol.ex 20.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master      #26   +/-   ##
=========================================
  Coverage          ?   51.85%           
=========================================
  Files             ?        9           
  Lines             ?       27           
  Branches          ?        0           
=========================================
  Hits              ?       14           
  Misses            ?       13           
  Partials          ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

# alog doesn't have entry_id as a required_field, it just ensures that an
# entry_id is added to the changeset.
# see https://github.com/dwyl/alog/blob/master/lib/alog.ex#L112
# :entry_id
Copy link
Member

Choose a reason for hiding this comment

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

we need make a couple of data transformation tables for this example to explain what the primary key of each record is so that:
a) each revision of the address record is unique
b) the address for a given person can be looked up by ID and we can see the history of the record.

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

Successfully merging this pull request may close these issues.

3 participants