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

Release 0.18.1 #272

Merged
merged 41 commits into from
Apr 21, 2020
Merged

Release 0.18.1 #272

merged 41 commits into from
Apr 21, 2020

Conversation

gavinballard
Copy link
Member

gavinballard and others added 30 commits April 16, 2020 11:58
@gavinballard
rbenv -> asdf
c98ca8d
@gavinballard
Add Node version to initialise.sh
9c3e757
@gavinballard
Add TriggerUsage model to track status of flow triggers
ddbd81c
@gavinballard
Add UpdateTriggerUsage service method
130a90d
@gavinballard
Add trigger usage controller
7d5bd02
@gavinballard
Fix long index name
2dca220
@gavinballard
Update test schema
b2bff82
@gavinballard
Change service call to expect a parsed Time object
4f7618e
@gavinballard
Add test for trigger usage service
36620a0
@gavinballard
Add controller tests for trigger usage updates
0866152
@gavinballard
Cleanup
a79275b
@gavinballard
Add (failing) test for skipping triggers
10a3df3
@gavinballard
Update processing to skip unused triggers
d1c5b5e
@gavinballard
Add additional flow trigger tests
d1c0abd
@gavinballard
Add Flow Trigger Usage info to README etc
c130f09
@gavinballard
Parse shop timezone from iana_timezone attribute
3e7aad2
@gavinballard
Tweak dependencies
9aaa713
@gavinballard
Update PULL_REQUEST_TEMPLATE.md
1e92991
@gavinballard
Update CHANGELOG.md
fd4a1e4
@gavinballard
Update UPGRADING.md
743b369
@gavinballard
Remove test matrix line from PR template
20c4e04
@gavinballard
Add nodejs version to .tool-versions
32fc33b
@gavinballard
Add nodejs to .tool-versions template
165ad88
@gavinballard
Fix some tests for new timezone method
b445f24
@tomcdisco
use sass-rails version 6 to match rails version
c0753de
@tomcdisco
sass-rails 6 seems to require manifest.js
af3d05a
@tomcdisco
Merge pull request #270 from discolabs/bugfix/rails6_incompatible_gem…
777c62c 
…_when_initialising_new_rails_app

Bugfix/rails6 incompatible gem when initialising new rails app
@gavinballard
Merge branch 'develop' into add-flow-usage
b496a2d
@gavinballard
Add default jobs and concerns for GDPR webhooks
813e006
@gavinballard
Add ability to specify fields for webhooks
5a9300f
pillep
pillep approved these changes Apr 20, 2020
View reviewed changes
Copy link
Contributor

@pillep pillep left a comment

Choose a reason for hiding this comment

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

🚀

tomcdisco
tomcdisco approved these changes Apr 21, 2020
View reviewed changes
Copy link
Contributor

@tomcdisco tomcdisco left a comment

Choose a reason for hiding this comment

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

LGTM. just tested that initialise.sh script worked and the resulting app was ok.

@gavinballard gavinballard merged commit 1440f6b into master Apr 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tomcdisco tomcdisco tomcdisco approved these changes

@pillep pillep pillep approved these changes

@zero51 zero51 zero51 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@gavinballard @zero51 @pillep @tomcdisco