Skip to content

Comments

fix: improve responsiveness of greet method in Motoko project template#2345

Merged
smallstepman merged 5 commits intomasterfrom
greet-query
Aug 15, 2022
Merged

fix: improve responsiveness of greet method in Motoko project template#2345
smallstepman merged 5 commits intomasterfrom
greet-query

Conversation

@matthewhammer
Copy link
Contributor

Description

greet is marked as an update call, but it performs no state updates. It can be faster as a query call.

@matthewhammer matthewhammer requested a review from a team as a code owner July 14, 2022 17:05
@matthewhammer matthewhammer changed the title speed up greet example's response time. Fix: speed up greet example's response time. Jul 14, 2022
@matthewhammer matthewhammer changed the title Fix: speed up greet example's response time. Fix: improve responsiveness of Motoko example. Jul 14, 2022
@matthewhammer matthewhammer changed the title Fix: improve responsiveness of Motoko example. fix: improve responsiveness of Motoko example. Jul 14, 2022
ghost
ghost previously requested changes Jul 14, 2022
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

Would you please add an entry to the changelog?

@ghost ghost dismissed their stale review July 15, 2022 20:01

I'm on PTO next week, so don't want to prevent this from being merged.

@smallstepman smallstepman requested a review from a user August 15, 2022 15:47
@smallstepman smallstepman changed the title fix: improve responsiveness of Motoko example. fix: improve responsiveness of greet method in Motoko project template Aug 15, 2022
@smallstepman smallstepman enabled auto-merge (squash) August 15, 2022 15:49
@smallstepman smallstepman merged commit 24b8f68 into master Aug 15, 2022
@smallstepman smallstepman deleted the greet-query branch August 15, 2022 16:26
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