{"action":"created","issue":{"url":"https://api.github.com/repos/mercadopago/px-android/issues/1227","repository_url":"https://api.github.com/repos/mercadopago/px-android","labels_url":"https://api.github.com/repos/mercadopago/px-android/issues/1227/labels{/name}","comments_url":"https://api.github.com/repos/mercadopago/px-android/issues/1227/comments","events_url":"https://api.github.com/repos/mercadopago/px-android/issues/1227/events","html_url":"https://github.com/mercadopago/px-android/pull/1227","id":359471881,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE0OTU0Mjk1","number":1227,"title":"[hotfix] Message queue para procesadora de pagos.","user":{"login":"zehemz","id":7827612,"node_id":"MDQ6VXNlcjc4Mjc2MTI=","avatar_url":"https://avatars0.githubusercontent.com/u/7827612?v=4","gravatar_id":"","url":"https://api.github.com/users/zehemz","html_url":"https://github.com/zehemz","followers_url":"https://api.github.com/users/zehemz/followers","following_url":"https://api.github.com/users/zehemz/following{/other_user}","gists_url":"https://api.github.com/users/zehemz/gists{/gist_id}","starred_url":"https://api.github.com/users/zehemz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zehemz/subscriptions","organizations_url":"https://api.github.com/users/zehemz/orgs","repos_url":"https://api.github.com/users/zehemz/repos","events_url":"https://api.github.com/users/zehemz/events{/privacy}","received_events_url":"https://api.github.com/users/zehemz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-09-12T13:07:23Z","updated_at":"2018-09-12T14:44:48Z","closed_at":null,"author_association":"COLLABORATOR","pull_request":{"url":"https://api.github.com/repos/mercadopago/px-android/pulls/1227","html_url":"https://github.com/mercadopago/px-android/pull/1227","diff_url":"https://github.com/mercadopago/px-android/pull/1227.diff","patch_url":"https://github.com/mercadopago/px-android/pull/1227.patch"},"body":"Antiguamente era necesario conocer el ciclo de vida de la procesadora de pago para poder llamar a los callbacks pertinentes.\r\n\r\nEsto requería adicionalmente contar con un caché en cada procesadora para manejar los casos de rotación, background, etc.\r\n\r\nLa solución propuesta es un boceto (o no...) de una posible implementación stateless que permite al integrador abstraerse del ciclo de vida de PX cuando procesa pagos no visuales.\r\n\r\nLa idea detrás de esto es que se encolen los mensajes en un mem cache en forma de queue. Al recibir los mensajes intentamos procesarlos inmediatamente, de no ser posible reintentamos cuando la actividad vuelve a resumirse."},"comment":{"url":"https://api.github.com/repos/mercadopago/px-android/issues/comments/420675271","html_url":"https://github.com/mercadopago/px-android/pull/1227#issuecomment-420675271","issue_url":"https://api.github.com/repos/mercadopago/px-android/issues/1227","id":420675271,"node_id":"MDEyOklzc3VlQ29tbWVudDQyMDY3NTI3MQ==","user":{"login":"codecov-io","id":8655789,"node_id":"MDQ6VXNlcjg2NTU3ODk=","avatar_url":"https://avatars0.githubusercontent.com/u/8655789?v=4","gravatar_id":"","url":"https://api.github.com/users/codecov-io","html_url":"https://github.com/codecov-io","followers_url":"https://api.github.com/users/codecov-io/followers","following_url":"https://api.github.com/users/codecov-io/following{/other_user}","gists_url":"https://api.github.com/users/codecov-io/gists{/gist_id}","starred_url":"https://api.github.com/users/codecov-io/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/codecov-io/subscriptions","organizations_url":"https://api.github.com/users/codecov-io/orgs","repos_url":"https://api.github.com/users/codecov-io/repos","events_url":"https://api.github.com/users/codecov-io/events{/privacy}","received_events_url":"https://api.github.com/users/codecov-io/received_events","type":"User","site_admin":false},"created_at":"2018-09-12T14:44:47Z","updated_at":"2018-09-12T14:44:47Z","author_association":"NONE","body":"# [Codecov](https://codecov.io/gh/mercadopago/px-android/pull/1227?src=pr&el=h1) Report\n> Merging [#1227](https://codecov.io/gh/mercadopago/px-android/pull/1227?src=pr&el=desc) into [hotfix/4.0.2](https://codecov.io/gh/mercadopago/px-android/commit/38c7d5abd854b3be9dc90a5f59adb1ba40e9a19a?src=pr&el=desc) will **decrease** coverage by `0.01%`.\n> The diff coverage is `0%`.\n\n[![Impacted file tree graph](https://codecov.io/gh/mercadopago/px-android/pull/1227/graphs/tree.svg?width=650&token=ksLU5tHdUS&height=150&src=pr)](https://codecov.io/gh/mercadopago/px-android/pull/1227?src=pr&el=tree)\n\n```diff\n@@                Coverage Diff                @@\n##             hotfix/4.0.2   #1227      +/-   ##\n=================================================\n- Coverage            2.58%   2.57%   -0.02%     \n  Complexity            125     125              \n=================================================\n  Files                 365     365              \n  Lines               14948   15022      +74     \n  Branches             1310    1312       +2     \n=================================================\n  Hits                  387     387              \n- Misses              14536   14610      +74     \n  Partials               25      25\n```\n\n\n| [Impacted Files](https://codecov.io/gh/mercadopago/px-android/pull/1227?src=pr&el=tree) | Coverage Δ | Complexity Δ | |\n|---|---|---|---|\n| [...ternal/callbacks/PaymentServiceHandlerWrapper.java](https://codecov.io/gh/mercadopago/px-android/pull/1227/diff?src=pr&el=tree#diff-cHgtY2hlY2tvdXQvc3JjL21haW4vamF2YS9jb20vbWVyY2Fkb3BhZ28vYW5kcm9pZC9weC9pbnRlcm5hbC9jYWxsYmFja3MvUGF5bWVudFNlcnZpY2VIYW5kbGVyV3JhcHBlci5qYXZh) | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |\n| [...ndroid/px/internal/features/CheckoutPresenter.java](https://codecov.io/gh/mercadopago/px-android/pull/1227/diff?src=pr&el=tree#diff-cHgtY2hlY2tvdXQvc3JjL21haW4vamF2YS9jb20vbWVyY2Fkb3BhZ28vYW5kcm9pZC9weC9pbnRlcm5hbC9mZWF0dXJlcy9DaGVja291dFByZXNlbnRlci5qYXZh) | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |\n| [...atures/plugins/PaymentProcessorPluginActivity.java](https://codecov.io/gh/mercadopago/px-android/pull/1227/diff?src=pr&el=tree#diff-cHgtY2hlY2tvdXQvc3JjL21haW4vamF2YS9jb20vbWVyY2Fkb3BhZ28vYW5kcm9pZC9weC9pbnRlcm5hbC9mZWF0dXJlcy9wbHVnaW5zL1BheW1lbnRQcm9jZXNzb3JQbHVnaW5BY3Rpdml0eS5qYXZh) | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |\n| [...d/px/internal/features/onetap/OneTapPresenter.java](https://codecov.io/gh/mercadopago/px-android/pull/1227/diff?src=pr&el=tree#diff-cHgtY2hlY2tvdXQvc3JjL21haW4vamF2YS9jb20vbWVyY2Fkb3BhZ28vYW5kcm9pZC9weC9pbnRlcm5hbC9mZWF0dXJlcy9vbmV0YXAvT25lVGFwUHJlc2VudGVyLmphdmE=) | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |\n| [...id/px/internal/features/onetap/OneTapFragment.java](https://codecov.io/gh/mercadopago/px-android/pull/1227/diff?src=pr&el=tree#diff-cHgtY2hlY2tvdXQvc3JjL21haW4vamF2YS9jb20vbWVyY2Fkb3BhZ28vYW5kcm9pZC9weC9pbnRlcm5hbC9mZWF0dXJlcy9vbmV0YXAvT25lVGFwRnJhZ21lbnQuamF2YQ==) | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |\n| [...android/px/internal/datasource/PaymentService.java](https://codecov.io/gh/mercadopago/px-android/pull/1227/diff?src=pr&el=tree#diff-cHgtY2hlY2tvdXQvc3JjL21haW4vamF2YS9jb20vbWVyY2Fkb3BhZ28vYW5kcm9pZC9weC9pbnRlcm5hbC9kYXRhc291cmNlL1BheW1lbnRTZXJ2aWNlLmphdmE=) | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |\n| [...ndroid/px/configuration/AdvancedConfiguration.java](https://codecov.io/gh/mercadopago/px-android/pull/1227/diff?src=pr&el=tree#diff-cHgtY2hlY2tvdXQvc3JjL21haW4vamF2YS9jb20vbWVyY2Fkb3BhZ28vYW5kcm9pZC9weC9jb25maWd1cmF0aW9uL0FkdmFuY2VkQ29uZmlndXJhdGlvbi5qYXZh) | `0% <0%> (ø)` | `0% <0%> (ø)` | :arrow_down: |\n| [...ternal/datasource/MercadoPagoPaymentProcessor.java](https://codecov.io/gh/mercadopago/px-android/pull/1227/diff?src=pr&el=tree#diff-cHgtY2hlY2tvdXQvc3JjL21haW4vamF2YS9jb20vbWVyY2Fkb3BhZ28vYW5kcm9pZC9weC9pbnRlcm5hbC9kYXRhc291cmNlL01lcmNhZG9QYWdvUGF5bWVudFByb2Nlc3Nvci5qYXZh) | `0% <0%> (ø)` | `0% <0%> (ø)` | :arrow_down: |\n| [...dopago/android/px/internal/view/ButtonPrimary.java](https://codecov.io/gh/mercadopago/px-android/pull/1227/diff?src=pr&el=tree#diff-cHgtY2hlY2tvdXQvc3JjL21haW4vamF2YS9jb20vbWVyY2Fkb3BhZ28vYW5kcm9pZC9weC9pbnRlcm5hbC92aWV3L0J1dHRvblByaW1hcnkuamF2YQ==) | `0% <0%> (ø)` | `0% <0%> (ø)` | :arrow_down: |\n\n------\n\n[Continue to review full report at Codecov](https://codecov.io/gh/mercadopago/px-android/pull/1227?src=pr&el=continue).\n> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)\n> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`\n> Powered by [Codecov](https://codecov.io/gh/mercadopago/px-android/pull/1227?src=pr&el=footer). Last update [38c7d5a...5681914](https://codecov.io/gh/mercadopago/px-android/pull/1227?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).\n"}}
{"actor":{"display_login":"codecov-io"}}