Skip to content

auto-gen: SDK changes (a51c84a5) - #8

Draft
luismeli10 wants to merge 3 commits into
masterfrom
auto-gen/a51c84a5-generic
Draft

luismeli10 wants to merge 3 commits into
masterfrom
auto-gen/a51c84a5-generic

Conversation

@luismeli10

Copy link
Copy Markdown
Owner

Automated code generation.

generation_id: a51c84a5-8d00-401d-8930-7395dc8218b4

Review required before merge.

@luismeli10

Copy link
Copy Markdown
Owner Author

Automated reviewer warnings

This pull request was created with the following advisory findings. Review them before merging:

  • Planner expanded the selected repository file scope to modify: tests/MercadoPago/Resources/Mocks/Response/Order/order_checkout_pro.json, src/MercadoPago/Resources/Order/Payment.php, src/MercadoPago/Resources/Payment/Order.php, examples/Adva
  • La solicitud menciona un «link adjunto», pero no se incluye ningún enlace ni adjunto en el material revisable. Por ello no es posible verificar que el ejemplo siga ese recurso ni que se haya incorporado dicho enlace.

@luismeli10

Copy link
Copy Markdown
Owner Author

Automated reviewer warnings

This pull request was created with the following advisory findings. Review them before merging:

  • replacement or rename request has no explicit old-to-new contract
  • Planner expanded the selected repository file scope to modify: tests/MercadoPago/Resources/Mocks/Response/Order/order_checkout_pro.json, tests/MercadoPago/Resources/Mocks/Response/MerchantOrder/merchant_order_base.json, tests/MercadoPago/Re
  • forbidden symbol remains in generated files: PreferenceClient
  • forbidden symbol remains in generated files: Preference
  • required symbol is not proven by loaded repository context: $order->type = "online"
  • required symbol is not proven by loaded repository context: $order->processing_mode = "manual"
  • README.md: permanece el encabezado/paso obsoleto ### Step 3: Create customer's preference before proceeding to Checkout Pro page, pese a que CHG-001 prohíbe Preference en este archivo. Debe sustituirse por un paso de creación de la orde
  • README.md: tras los nuevos imports (RequestOptions, OrderClient, Order), el diff no incorpora el código que instancia OrderClient, configura RequestOptions/autenticación ni construye/crea una Order. Por tanto falta el reemplazo
  • README.md: la frase nueva dice que se debe redirigir a checkout_url, pero el fragmento documentado sigue anunciando que se creará una preference. Esto contradice el comportamiento de la nueva API.
  • README.md: el plan CHG-001 exige eliminar las referencias prohibidas de Preference en el alcance indicado. La ocurrencia literal preference en el encabezado de Step 3 sigue siendo evidencia obsoleta.
  • No hay evidencia en el diff de que tests/MercadoPago/Resources/Mocks/Response/Order/order_checkout_pro.json haya sido actualizado para cumplir CHG-003, que prohíbe preference_id, Preference y preferences. El diff final no muestra es
  • examples/Order/CreateCheckoutPRO.php: reemplazar las salidas de ID/estado/URL por header("Location: " . $order->checkout_url); exit; cambia el ejemplo hacia una redirección real y es coherente con la solicitud, pero debe verificarse en el

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant