-
Notifications
You must be signed in to change notification settings - Fork 5
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
PR to track downstream changes in a fork #12
Open
AndrewDryga
wants to merge
34
commits into
Nebo15:master
Choose a base branch
from
valiot:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
c9d5b4a
Set PR template
valiotbot 21aee94
remove uuid change to normal id
gdavila09 cd269fa
mix format
acrogenesis cf3d6f7
working tests
acrogenesis e179f9b
Merge pull request #1 from valiot/gdavila/TEAPP-30-uuid_to_id
acrogenesis bc17834
update deps
gdavila09 2ff19c1
removed updated at false
gdavila09 3b70013
Merge pull request #2 from valiot/gdavila/update-ecto
acrogenesis bade652
delete and log function
gdavila09 b1abcc1
Merge pull request #3 from valiot/gdavila/TEAPP-23-delete-and-log
acrogenesis f849298
remove unused variable
acrogenesis 3df7d04
upsert in ecto trail
gdavila09 6bb34a0
remove dogma, update deps
acrogenesis 28f5a56
fix Multi.run deprecation warning
acrogenesis 8ee7d94
Merge pull request #4 from valiot/gdavila/TEAPP-70-upsert-log
acrogenesis b9348f2
passing tests
acrogenesis 34f529a
log the type of change
gdavila09 d215432
changed type from created to insert
gdavila09 266442b
change enum to insert and test upsert
gdavila09 e5b6300
fix test
gdavila09 e93bc16
Merge pull request #5 from valiot/gdavila/TEAPP-84-log-type-change
acrogenesis eb43295
update de readme
gdavila09 4683628
Merge pull request #6 from valiot/gdavila/TEAPP-84-log-type-change
acrogenesis c36819b
rename ChangeEnum
acrogenesis bd1e1c0
update deps
acrogenesis 641c2b2
update dialyxir
acrogenesis d2b4b85
redact custom fields from changeset
acrogenesis c926fe0
solve breaking change, redacted_fields not required
acrogenesis 2c9cf3f
Merge pull request #8 from valiot/maintenance
gdavila09 f28fc13
change data structure for delete changesets
afloram d04bb25
update dependencies
afloram 32cba31
Merge pull request #9 from valiot/alex/VAPP-136-format-for-delete-cha…
acrogenesis eec819c
Corral/log only (#10)
doctorcorral a8d8d77
remove unused vars
acrogenesis File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
# Used by "mix format" | ||
[ | ||
line_length: 110, | ||
inputs: ["mix.exs", "{config,lib,test}/**/*.{ex,exs}"] | ||
] | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,3 @@ | ||
use Mix.Config | ||
|
||
config :ecto_trail, | ||
table_name: "audit_log" | ||
config :ecto_trail, table_name: "audit_log", redacted_fields: [:password] |
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There should be a final \n at the end of each file.