Skip to content

feat(aws): add iot event schema#8177

Merged
medikoo merged 2 commits into
serverless:masterfrom
pgrzesik:add-aws-iot-event-schema
Sep 3, 2020
Merged

feat(aws): add iot event schema#8177
medikoo merged 2 commits into
serverless:masterfrom
pgrzesik:add-aws-iot-event-schema

Conversation

@pgrzesik

@pgrzesik pgrzesik commented Sep 2, 2020

Copy link
Copy Markdown
Contributor

Add IoT Event Schema + minor cleanup

Closes: #8030

@pgrzesik
pgrzesik force-pushed the add-aws-iot-event-schema branch from a46d0fb to 2c97dc7 Compare September 2, 2020 15:01
@codecov-commenter

codecov-commenter commented Sep 2, 2020

Copy link
Copy Markdown

Codecov Report

Merging #8177 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #8177      +/-   ##
==========================================
- Coverage   88.22%   88.21%   -0.01%     
==========================================
  Files         248      248              
  Lines        9459     9460       +1     
==========================================
  Hits         8345     8345              
- Misses       1114     1115       +1     
Impacted Files Coverage Δ
...ib/plugins/aws/package/compile/events/iot/index.js 100.00% <100.00%> (ø)
lib/classes/Service.js 85.48% <0.00%> (-0.81%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3a0f2fc...c7c43e4. Read the comment docs.

@medikoo medikoo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @pgrzesik that looks great! I have just one very minor style remark

const ruleName = event.iot.name ? event.iot.name.replace(/\r?\n/g, '') : '';
const awsIotSqlVersion = event.iot.sqlVersion
? event.iot.sqlVersion.replace(/\r?\n/g, '')
: '';

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's use null as no value

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done ✔️

@pgrzesik
pgrzesik requested a review from medikoo September 3, 2020 07:10

@medikoo medikoo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you @pgrzesik !

@medikoo
medikoo merged commit e55fc36 into serverless:master Sep 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Config schema: Define AWS "iot" function event properties

3 participants