Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@JamieD1
Copy link
Contributor

@JamieD1 JamieD1 commented May 10, 2020

Intent of your Pull Request

Fixes Agent ID's Forging,

Also adds bank account changing

Fixes #8405

Changelog

🆑
rscadd: Adds bank accounts to agent ids
bugfix: Fixes Agent ID Forging
/:cl:

@yogstation13-bot yogstation13-bot added the Feature This adds new content to the game label May 10, 2020
@swissloaf swissloaf added the Awaiting - Action - Maintainer This PR is awaiting an action from a maintainer label May 10, 2020
@boodaliboo
Copy link
Contributor

Intent of your Pull Request

Fixes Agent ID's Forging,

Also adds bank account changing

Fixes #8405

Changelog

🆑
rscadd: Adds bank accounts to agent ids
bugfix: Fixes Agent ID Forging
/🆑

how does this adding bank accounts work does it take the buyers bank does it take their set names bank?

@SomeguyManperson
Copy link
Contributor

image

@JamieD1
Copy link
Contributor Author

JamieD1 commented May 10, 2020

Intent of your Pull Request

Fixes Agent ID's Forging,
Also adds bank account changing
Fixes #8405

Changelog

🆑
rscadd: Adds bank accounts to agent ids
bugfix: Fixes Agent ID Forging
/🆑

how does this adding bank accounts work does it take the buyers bank does it take their set names bank?

It random makes a bank account on spawn and it means you can change it via forging.

Copy link
Contributor

@fluffe9911 fluffe9911 left a comment

Choose a reason for hiding this comment

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

I hate this review system

@fluffe9911 fluffe9911 merged commit b580b31 into yogstation13:master May 19, 2020
@JamieD1 JamieD1 deleted the Agent-ID-Fix branch June 4, 2020 23:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Awaiting - Action - Maintainer This PR is awaiting an action from a maintainer Feature This adds new content to the game

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Agent IDs dont forge a 2nd time after they have a name on them.

6 participants