Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Keep the Bank Banking. #3

Open
RTippin opened this issue Jun 30, 2022 · 6 comments
Open

Keep the Bank Banking. #3

RTippin opened this issue Jun 30, 2022 · 6 comments
Labels
cringe You just posted cringe enhancement New feature or request
Milestone

Comments

@RTippin
Copy link

RTippin commented Jun 30, 2022

It appears to me that once a bank in one remote location is updated, the source-of-truth-bank here on github does not currently reflect this change. Might I suggest adding the logic under

file_put_contents(__DIR__.'/bank/quote'.$next_quote_no.'.mark', $body);
to sync the banks?

file_put_contents(__DIR__.'/bank/quote'.$next_quote_no.'.mark', $body);
`git add . && git commit -m "sync amazing bank" && git push`; //1337 inline php exec

Then you can continue being a teapot and feel snug knowing the banks are banking.

@jorqensen jorqensen added the enhancement New feature or request label Jun 30, 2022
@jorqensen
Copy link
Owner

Thanks for the suggestion, I believe we can add this to a 1.2 roadmap, maybe push to 2.0 release for potential breaking changes.

@arkoe thoughts?

@jorqensen jorqensen added this to the 2.0 milestone Jun 30, 2022
@arkoe
Copy link
Collaborator

arkoe commented Jun 30, 2022

Interesting

@RTippin
Copy link
Author

RTippin commented Jul 1, 2022

Interesting

@arkoe I'm enthralled by your position regarding these titillating discussions!
FeelsJorqensenMan

@Smoggert
Copy link
Contributor

We could consider putting this on Web3, where if the majority of pull-requests tries to add the same quote to the bank it gets passed automatically.

#blockchains #wayofthefuture #wayofthefuture #wayofthefuture #wayofthefuture

@RTippin
Copy link
Author

RTippin commented Jul 10, 2022

@Smoggert If Web3 is the path, then indeed @jorqensen needs to be assigned to this enhancement. He knows de wae.

@jorqensen
Copy link
Owner

@arkoe we need alternative soluton for this cba

@jorqensen jorqensen removed this from the 2.0 milestone Jul 11, 2022
@arkoe arkoe added this to the 3.0 milestone Jul 11, 2022
@jorqensen jorqensen added the cringe You just posted cringe label Jul 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cringe You just posted cringe enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants