Skip to content
This repository has been archived by the owner on Aug 28, 2020. It is now read-only.

Add Settings Sync Code (Auto Sync) #1012

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

UnseenFaith
Copy link
Contributor

Description of the PR

Adds back some old functionality from old Klasa (syncing users upon fetch via KlasaUserStore) and some of the other functionality we wanted from being able to override actions.

This needs my other PR on core to merge before this will build for Action#run to be async.

This also needs some discussion about User#settings, it seems not as trivial as guilds to sync.

Changes Proposed in this Pull Request (List new items in CHANGELOG.MD)

  • KlasaClient#users: UserStore => KlasaUserStore
  • KlasaUserStore#fetch => syncs settings

Semver Classification

  • This PR only includes documentation or non-code changes.
  • This PR fixes a bug and does not change the (intended) framework interface.
  • This PR adds methods or properties to the framework interface.
  • This PR removes or renames methods or properties in the framework interface.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants