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

Implement the LinkShare #90

Merged
merged 1 commit into from
Dec 8, 2022
Merged

Implement the LinkShare #90

merged 1 commit into from
Dec 8, 2022

Conversation

kkocdko
Copy link
Contributor

@kkocdko kkocdko commented Dec 7, 2022

No description provided.

@kkocdko
Copy link
Contributor Author

kkocdko commented Dec 7, 2022

Usage:

CLIENT
    .send_group_link_share(
        e.inner.group_code,
        LinkShare {
            title: "Bing".into(),
            summary: Some("Summary here".into()),
            brief: Some("Brief here".into()),
            picture_url: Some("https://pic.ugcimg.cn/945bdbdc0f0db3e92010c47520693283/jpg1".into()),
            url: "https://bing.com".into(),
        },
    )
    .await
    .unwrap();

@kkocdko
Copy link
Contributor Author

kkocdko commented Dec 7, 2022

Thanks to takayama-lily/oicq#437 !

@kkocdko
Copy link
Contributor Author

kkocdko commented Dec 8, 2022

Seems that this function is suteable to placed near the MusicShare? Just merge them into one file?

Maybe next PR. Feel free to merge this now.

Copy link
Contributor Author

@kkocdko kkocdko left a comment

Choose a reason for hiding this comment

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

Interesting.

@lz1998 lz1998 merged commit c3ec94e into lz1998:master Dec 8, 2022
@kkocdko kkocdko deleted the link_share branch December 8, 2022 10:04
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