[1] Cleanup unused field, fix and define action payload type

[MrOrz]

• refactor: remove issuedAt
  • issuedAt logic was initially introduced before 2018 (https://github.com/cofacts/rumors-line-bot/pull/113/files) and was long replaced by sessionId
  • This PR removes the remaining issuedAt
• fix(chatbotState): replace PostbackEvent with actual type used in handlers
  • The new type describes the weird payload event (type: payload + input) that the chatbot uses
  • Manyas ChatbotEvent can be removed as the current ChatbotEvent can precisely describe the event being passed
• refactor(webhook): Read / write of postback data is protected by type
  • Define the payload action that is currently being used
• refactor(gql): narrow the type of GraphQL response
  • Better data & error that matches the actual scenario

[coveralls]

Pull Request Test Coverage Report for Build 6535983227

• 9 of 14 (64.29%) changed or added relevant lines in 5 files are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage decreased (-0.06%) to 90.435%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/webhook/index.ts	0	5	0.0%

Files with Coverage Reduction	New Missed Lines	%
src/webhook/index.ts	1	49.61%

Totals
Change from base Build 6527795742:	-0.06%
Covered Lines:	982
Relevant Lines:	1057

---

💛 - Coveralls

[MrOrz]

Current dependencies on/for this PR:

• master
  • PR [1] Cleanup unused field, fix and define action payload type #366 👈
    • PR [2] Convert webhook handlers to Typescript #365
      • PR [3] remove selectedArticleText and selectedReplyId from context #368

This comment was auto-generated by Graphite.

[MrOrz]

Merge activity

• Oct 18, 2:04 PM: @MrOrz started a stack merge that includes this pull request via Graphite.
• Oct 18, 2:04 PM: @MrOrz merged this pull request with Graphite.