-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat(GiniHealthSDK): Payment component documentation and integration guide IPC-98 #453
feat(GiniHealthSDK): Payment component documentation and integration guide IPC-98 #453
Conversation
- Added documentation for the classes, methods, and variables that are public from the payment component flow - Did some small cleaning IPC-98
…nent Added 2 photos of the payment component: The payment component view and the payment component screen IPC-98
…w initialization IPC-98
…documenting_payment_component
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @razvancapra for this pr.
I left some suggestions. We can discuss them if you need.
.../GiniHealthSDK/Sources/GiniHealthSDK/Core/PaymentComponent/PaymentComponentsController.swift
Outdated
Show resolved
Hide resolved
.../GiniHealthSDK/Sources/GiniHealthSDK/Core/PaymentComponent/PaymentComponentsController.swift
Outdated
Show resolved
Hide resolved
.../GiniHealthSDK/Sources/GiniHealthSDK/Core/PaymentComponent/PaymentComponentsController.swift
Outdated
Show resolved
Hide resolved
.../GiniHealthSDK/Sources/GiniHealthSDK/Core/PaymentComponent/PaymentComponentsController.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@razvancapra thank you very much for the documentation! 👏🏻
Please, ignore for my comments about removing the empty lines 🙏
.../GiniHealthSDK/Sources/GiniHealthSDK/Core/PaymentComponent/PaymentComponentsController.swift
Outdated
Show resolved
Hide resolved
.../GiniHealthSDK/Sources/GiniHealthSDK/Core/PaymentComponent/PaymentComponentsController.swift
Outdated
Show resolved
Hide resolved
.../GiniHealthSDK/Sources/GiniHealthSDK/Core/PaymentComponent/PaymentComponentsController.swift
Outdated
Show resolved
Hide resolved
…documenting_payment_component
Formatting suggestions and rephrasing IPC-98
…from review Fixing suggestions from review. Rephrasing and adequate language IPC-98
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There're a few small things left for the keeping the format:
- Please add an empty line before and after code snippet.
- Let's remove
####
before the numbers 🙏
…documenting_payment_component
…documenting_payment_component
…documenting_payment_component
…documenting_payment_component
…documenting_payment_component
…documenting_payment_component
…documenting_payment_component
…nt provider IPC-98
.../GiniHealthSDK/Sources/GiniHealthSDK/Core/PaymentComponent/PaymentComponentsController.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@razvancapra looks good,
left a tiny suggestion.
…Controller IPC-98 Co-authored-by: Nadzeya Karaban <[email protected]>
Integration.md