Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Misc. Changes #52

Merged
merged 1 commit into from Aug 29, 2021
Merged

Misc. Changes #52

merged 1 commit into from Aug 29, 2021

Conversation

Citymonstret
Copy link
Member

  • Add interact() to Click
  • Switch click context over from a cancellation boolean to a status enum
  • allow for "chained" click handlers
  • update the example plugin

@Citymonstret Citymonstret force-pushed the feature/interact-clicks branch 2 times, most recently from 0727b5b to 03a9776 Compare August 29, 2021 14:43
Copy link
Member

@broccolai broccolai left a comment

Choose a reason for hiding this comment

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

LGTM.
Except the generics :')

- Add `interact()` to Click
- Switch click context over from a cancellation boolean to a status enum
- allow for "chained" click handlers
- update the example plugin
@Citymonstret Citymonstret merged commit c444deb into master Aug 29, 2021
@Citymonstret Citymonstret deleted the feature/interact-clicks branch August 29, 2021 14:51
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.

None yet

2 participants