Skip to content
This repository has been archived by the owner on Mar 13, 2023. It is now read-only.

feat: add client.owner_ids #588

Merged
merged 3 commits into from
Aug 9, 2022
Merged

feat: add client.owner_ids #588

merged 3 commits into from
Aug 9, 2022

Conversation

LordOfPolls
Copy link
Member

What type of pull request is this?

  • Non-breaking code change
  • Breaking code change
  • Documentation change/addition
  • Tests change

Description

Adds support for declaring owner ids in code. This allows users to add other users as owners, without creating a team.

Changes

  • Add client.owner_ids attribute
  • Update is_owner check to respect client.owner_ids
  • Add API-True owner to owner_ids on startup

Checklist

  • I've formatted my code with Black
  • I've ensured my code works on Python 3.10.x
  • I've tested my code

@LordOfPolls LordOfPolls added the New Feature::Library A new feature for the library label Aug 7, 2022
silasary
silasary suggested changes Aug 7, 2022
View reviewed changes
naff/models/naff/checks.py Outdated Show resolved Hide resolved
@LordOfPolls LordOfPolls requested a review from silasary August 7, 2022 09:13
@LordOfPolls LordOfPolls merged commit db54f08 into dev Aug 9, 2022
@LordOfPolls LordOfPolls deleted the owner_ids branch August 9, 2022 14:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@silasary silasary silasary approved these changes

Assignees
No one assigned
Labels
New Feature::Library A new feature for the library
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@LordOfPolls @silasary