[v2] Fix #436 View type missing optional id from response

[seratch]

Summary

This pull request is the v2 branch fix for #436 - cherry-picked from #437

Requirements (place an x in each [ ])

• I've read and understood the Contributing Guidelines and have done my best effort to follow them.
• I've read and agree to the Code of Conduct.

[codecov]

Codecov Report

Merging #438 into @slack/bolt@next will not change coverage by %.
The diff coverage is n/a.

@@                Coverage Diff                @@
##           @slack/bolt@next     #438   +/-   ##
=================================================
  Coverage             85.60%   85.60%           
=================================================
  Files                     7        7           
  Lines                   514      514           
  Branches                149      149           
=================================================
  Hits                    440      440           
  Misses                   50       50           
  Partials                 24       24           

---

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 3a1fc9e...cdd9e71. Read the comment docs.

[aoberoi]

@seratch instead of opening a second PR for the @slack/bolt@next branch, i think its easier to allow the PR to merge on master and then merge master into @slack/bolt@next.

@stevengill can you confirm this is how we want to handle this scenario?

[seratch]

Yes, either is fine to me. I just wanted to reduce y’all’s efforts on this considering this timing (last minute before v2 release)

[stevengill]

agree with @aoberoi!

Any PR that needs to hit the 1.x branch should be going to master and merged into next from master.

Any PR that is next specific can go straight to next

Closing this PR.