Skip to content

Conversation

@hch903
Copy link
Contributor

@hch903 hch903 commented Jun 2, 2020

get/set/delete user level persistent menu for context user

@hch903 hch903 force-pushed the context-persistent-menu branch from 98f37b1 to 2b4b7da Compare June 9, 2020 14:08
@chentsulin chentsulin changed the base branch from master to release/1.5 June 16, 2020 10:34
@chentsulin
Copy link
Collaborator

@hch903 Please rebase or master release/1.5 to get the latest MessengerClient.

@codecov-commenter
Copy link

codecov-commenter commented Jun 22, 2020

Codecov Report

Merging #790 into release/1.5 will increase coverage by 2.85%.
The diff coverage is 91.46%.

Impacted file tree graph

@@               Coverage Diff               @@
##           release/1.5     #790      +/-   ##
===============================================
+ Coverage        81.08%   83.94%   +2.85%     
===============================================
  Files              108      120      +12     
  Lines             4747     6190    +1443     
  Branches          1197     1534     +337     
===============================================
+ Hits              3849     5196    +1347     
- Misses             862      990     +128     
+ Partials            36        4      -32     
Impacted Files Coverage Δ
...s/bottender/src/cli/providers/messenger/profile.ts 83.05% <ø> (-3.56%) ⬇️
packages/bottender/src/console/ConsoleBot.ts 100.00% <ø> (ø)
packages/bottender/src/console/ConsoleConnector.ts 100.00% <ø> (ø)
packages/bottender/src/console/ConsoleContext.ts 100.00% <ø> (ø)
packages/bottender/src/messenger/MessengerBot.ts 100.00% <ø> (ø)
packages/bottender/src/slack/SlackEvent.ts 93.84% <ø> (-1.61%) ⬇️
packages/bottender/src/telegram/TelegramContext.ts 58.78% <ø> (+5.12%) ⬆️
packages/bottender/src/telegram/TelegramEvent.ts 97.50% <ø> (-0.65%) ⬇️
...kages/bottender/src/test-utils/SimulatedContext.ts 87.50% <ø> (-12.50%) ⬇️
packages/bottender/src/types.ts 100.00% <ø> (ø)
... and 139 more

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 718e61f...7f18a35. Read the comment docs.

false,
`getPersistentMenu: should not be called in context without session`
);
return;
Copy link
Collaborator

Choose a reason for hiding this comment

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

return null;

);
}

async getPersistentMenu(
Copy link
Collaborator

Choose a reason for hiding this comment

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

Sorry my bad. Now I'd name it to getUserPersistentMenu.

);
}

async setPersistentMenu(
Copy link
Collaborator

Choose a reason for hiding this comment

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

setUserPersistentMenu

);
}

async deletePersistentMenu(
Copy link
Collaborator

Choose a reason for hiding this comment

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

deleteUserPersistentMenu

@chentsulin chentsulin merged commit 50689fa into Yoctol:release/1.5 Jun 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants