Fix for "Not all roles synced. Discord API limit reached."

[TerranUlm]

1. Go to https://discordapp.com/developers/applications/me#top
2. Select the app for the sync
3. Select the "Bot" menu
4. Scroll down to "SERVER MEMBERS INTENT" and enable that option

done

[familypet]

MVP

[TerranUlm]

Another fix: replace all "X-RateLimit-..." strings to lowercase.

[familypet]

https://github.com/opt-natter/discord-mybb-sync/blob/9496d926f0ca860526578a7c41ad0dff94bb5ded/inc/plugins/discord_right_sync.php#L538

So from line 538, would look like this?

if (isset($response->headers['x-ratelimit-reset'])) //update reset time
        {
            $db->update_query("settings", array(
                'value' => (int)$response->headers['x-ratelimit-reset']
            ) , "name = 'drs_setting_ratelimit_reset'");
        }
        if ($response->headers['x-ratelimit-remaining'] == 0) //no more Querries allowed
        {
            return false;
        }
    }
}
return true;

}

[TerranUlm]

yes

[opt-natter]

Hi Terran,

are you sure about the lowecase? https://discord.com/developers/docs/topics/rate-limits

[TerranUlm]

Hi Natter,

Yes, the header returned in $response is all lower case. Might be due to the library for the rest calls. Since I'd changed it the timestamp is updated again.

[opt-natter]

Fixed in version 0.7.1