Request to merge epic/37-wire into master
0/1 thread resolved
added MR::Requires Changes scoped label
added 24 commits
- 31775e64 - (refactor): support hydrating the first batch of entities
- c490a207 - (chore): increase price to 1.25
- b8e8318e - (fix): onchain wire notifications now trigger
- fec067c2 - (feat): Retry queue for ElasticSearch
- 61b2956a - (feat): feed fixes including pinned posts for containers
- 4a65cc9f - (fix): failing spec tests
- 8f6ba21b - (fix): if no hydrated entities don't return entities
- 48d61f9d - (chore) crushing cockroach and cleaning up installer
- ac2d0e9f - Feat/email schema 187
- e4218a4e - (feat): whitehouse email template
- 1b6f465d - (feat): Client uploader
- fbc655fb - sprint/InterestingIguana: Blocked users can no longer send you a wire transactions #608 (closed)
- 0f3c1f46 - (chore): If mobile app then return generic urn
- 1c7c67cb - (fix): if faux activities then cast as such
- 12928e86 - (fix): reverse pinned posts array
- d1c86abe - (feat): transcoder now only listens for single message
- 6819a904 - (fix): 500 error on firehose
- 0e8b04fb - (fix): map to the entity
- 7c9aeeb4 - (fix): failing tests for firehose
- 941e6f9e - InterestingIguana Fixed blog edits not saving #593 (closed)
- 9026f4d1 - Resolve "(bug): publishing a blog that was saved as a draft or "unlisted" does...
- 0b951a37 - (fix): ensure arrays are sent to tags and not objects
- df35a293 - (fix): decode search query
- bf4cb175 - Sprint/InterestingIguana Add GitLab to settings example
Toggle commit listadded 1 commit
- de39c72f - (refactor): Add new Stripe Managers and refactor Wire to be abstracted from Paywall
added Squad::Blue scoped label
added 70 commits
- c2099aa6...0abf1410 - 68 commits from branch
master - 5d80ca91 - (chore): bump stripe version
- a852d273 - Merge branch 'master' of gitlab.com:minds/engine into epic/37-wire
- c2099aa6...0abf1410 - 68 commits from branch
marked the task ERC-20's as completed
marked the task ERC-20's as incomplete
added 2 commits
added 1 commit
- 4ef022e1 - (fix): resolve backend tests for new wire features
mentioned in merge request front!508
changed the description
added 76 commits
- 4ef022e1...c1dac85a - 75 commits from branch
master - 5b1950ec - Merge branch 'master' of gitlab.com:minds/engine into epic/37-wire-redux
- 4ef022e1...c1dac85a - 75 commits from branch
added 1 commit
- e091b442 - Merge branch 'master' of gitlab.com:minds/engine into epic/37-wire-redux
added 1 commit
- 31fdb7fc - (fix): email issues with wire, and other patches
added 9 commits
- 31fdb7fc...8e6942b6 - 6 commits from branch
master - 710463c4 - (chore): lint project before merge with master
- ff29e70c - Merge branch 'master' of gitlab.com:minds/engine into epic/37-wire
- f1f6cc1c - (fix): failing tests
Toggle commit list- 31fdb7fc...8e6942b6 - 6 commits from branch
unmarked as a Work In Progress
21 $paymentMethodsManager = new Stripe\PaymentMethods\Manager(); 22 $paymentMethods = $paymentMethodsManager->getList([ 23 'limit' => 12, 24 'user_guid' => $user->getGuid(), 25 ]); 26 27 return Factory::response([ 28 'paymentmethods' => Factory::exportable($paymentMethods), 29 ]); 30 } 31 32 public function post($pages) 33 { 34 $user = Session::getLoggedInUser(); 35 switch ($pages[0]) { 36 case 'apply': 
DeveloperThis isn't really RESTful and these switch statements on methods create the newsfeed/ switch monstrosities. If we POST to payment methods, we should be creating a payment method and the PUT should replace the resource by intent ID.
Granted, we have a lot of refactoring to do with the new routers, but this is a green field endpoint so it might be a good place to get more RESTful.
