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

fix: Add image picker config and fixes #96

Conversation

rohitsangwan01
Copy link
Contributor

@rohitsangwan01 rohitsangwan01 commented Jun 11, 2023

Description

Add options for image picker configurations like image width, height and image quality and a callback when image is picked , we can use this to modify image after picking, like to implement crop or compression or something else
fix issue caused by emoji_picker_flutter new version
exposed onChatListTap to ChatView

Checklist

  • The title of my PR starts with a [Conventional Commit] prefix (fix:, feat:, docs: etc).
  • I have followed the [Contributor Guide] when preparing my PR.
  • I have updated/added tests for ALL new/updated/fixed functionality.
  • I have updated/added relevant documentation in docs and added dartdoc comments with ///.
  • I have updated/added relevant examples in examples or docs.

Breaking Change?

  • Yes, this PR is a breaking change.
  • No, this PR is not a breaking change.

@tairoroberto
Copy link

Waiting for this PR, @rohitsangwan01. You need to update your version.

@rohitsangwan01
Copy link
Contributor Author

Waiting for this PR, @rohitsangwan01. You need to update your version.

Done 👍

@DhvanitVaghani
Copy link
Contributor

@rohitsangwan01 Can you please squash your commits into one?

@rohitsangwan01
Copy link
Contributor Author

rohitsangwan01 commented Aug 3, 2023

@DhvanitVaghani can you try Squash and Merge ?

Screenshot 2023-08-03 at 11 11 54 AM

@DhvanitVaghani DhvanitVaghani merged commit 32d14ab into SimformSolutionsPvtLtd:main Aug 3, 2023
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