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

added pins_get to get channel pins #4

Merged
merged 2 commits into from
Apr 5, 2021
Merged

added pins_get to get channel pins #4

merged 2 commits into from
Apr 5, 2021

Conversation

webtax-gh
Copy link
Member

@webtax-gh webtax-gh commented Apr 5, 2021

added pins_get to get channel pins

example test code:

bot.pins_get(event.msg->channel_id, [log, &bot](const dpp::confirmation_callback_t& completion) {
    dpp::message_map pins = std::get<dpp::message_map>(completion.value);
    log->debug("Pins got! number of pins={} http status={}", std::distance(std::begin(pins), std::end(pins)), completion.http_info.status);
});

this PR also includes fixes to channel_edit_permissions and channel_invites_get, as they were using the /api/channel endpoint instead of /api/channels

pins_get has been tested, the fixes to channel_edit_permissions and channel_invites_get have not been tested.

@webtax-gh webtax-gh changed the title git_pins to get channel pins get_pins to get channel pins Apr 5, 2021
@webtax-gh webtax-gh changed the title get_pins to get channel pins added pins_get to get channel pins Apr 5, 2021
@braindigitalis braindigitalis merged commit d4819b5 into brainboxdotcc:master Apr 5, 2021
braindigitalis pushed a commit that referenced this pull request Oct 27, 2021
braindigitalis pushed a commit that referenced this pull request Dec 17, 2021
braindigitalis pushed a commit that referenced this pull request Jan 9, 2022
braindigitalis pushed a commit that referenced this pull request May 27, 2022
step-security-bot pushed a commit to step-security-bot/DPP that referenced this pull request Sep 22, 2023
…y_remediation_1695375729

[StepSecurity] ci: Harden GitHub Actions
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.

2 participants