Skip to content

Dont use removed talentpool cache#2344

Merged
wookie184 merged 3 commits into
mainfrom
dont-use-removed-talentpool-cache
Nov 29, 2022
Merged

Dont use removed talentpool cache#2344
wookie184 merged 3 commits into
mainfrom
dont-use-removed-talentpool-cache

Conversation

@ChrisLovering
Copy link
Copy Markdown
Member

Closes #2340 Closes BOT-3AR

The nomination cache was recently removed, so accessing that no longer works.
@ChrisLovering ChrisLovering force-pushed the dont-use-removed-talentpool-cache branch from 2d4cdb0 to 7900bce Compare November 27, 2022 15:17
Copy link
Copy Markdown
Contributor

@wookie184 wookie184 left a comment

Choose a reason for hiding this comment

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

Looks good, thanks

@wookie184 wookie184 added t: bug Something isn't working a: information Related to information commands: (doc, help, information, reddit, site, tags) a: recruitment Related to recruitment: (talentpool) s: needs review Author is waiting for someone to review and approve labels Nov 27, 2022
Copy link
Copy Markdown
Contributor

@shtlrs shtlrs left a comment

Choose a reason for hiding this comment

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

Neat, thank you!

@wookie184 wookie184 enabled auto-merge November 29, 2022 20:26
@wookie184 wookie184 merged commit ffcb229 into main Nov 29, 2022
@wookie184 wookie184 deleted the dont-use-removed-talentpool-cache branch November 29, 2022 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: information Related to information commands: (doc, help, information, reddit, site, tags) a: recruitment Related to recruitment: (talentpool) s: needs review Author is waiting for someone to review and approve t: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AttributeError: 'TalentPool' object has no attribute 'cache'

3 participants