Skip to content

Conversation

@F-Shahali
Copy link
Collaborator

Reference Issues/PRs

#47

What does this implement/fix? Explain your changes.

Any other comments?

@F-Shahali F-Shahali added the enhancement New feature or request label Jan 29, 2025
@F-Shahali F-Shahali added this to the mybutton v0.6 milestone Jan 29, 2025
@F-Shahali F-Shahali requested a review from AHReccese January 29, 2025 19:42
@F-Shahali F-Shahali self-assigned this Jan 29, 2025
Copy link
Member

@sepandhaghighi sepandhaghighi left a comment

Choose a reason for hiding this comment

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

@F-Shahali Thank you for your efforts. Well done! 🔥

I have one comment:

  1. Please remove Custom Share from direct mode.

Copy link
Member

@AHReccese AHReccese left a comment

Choose a reason for hiding this comment

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

Thank you for your Efforts. Works as expected!

CHANGELOG.md Outdated
- `Custom Share` button
- Add `ServiceName` enum
### Changed
- Remove `Custom Share` from `direct` mode
Copy link
Member

Choose a reason for hiding this comment

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

Since Custom Share feature is just added in this PR, you don't need to write newly applied changes in this PR in the CHANGELOG.md.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

You are right, thanks.

@F-Shahali F-Shahali force-pushed the enhancement/custom-share branch from 7b70191 to c5525fe Compare February 1, 2025 19:31
Copy link
Member

@sepandhaghighi sepandhaghighi left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@F-Shahali F-Shahali merged commit be2f11a into dev Feb 3, 2025
3 checks passed
@F-Shahali F-Shahali deleted the enhancement/custom-share branch February 3, 2025 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants