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

New Payment Flow Challenge #372

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

WildCLown
Copy link

  • You have read the CONTRIBUTING.md
  • For Terraform changes, you have read the Terraform Naming Conventions
  • For Terraform changes, you have already executed the Terraform plan on this branch
  • The changes inside this pull request are organization-agnostic

Why?

Challenge for Backend Software Engineer (individual-assignment)

What?

1- It was refactored the Kotlin bootstrap.kt, separating classes, setting enum classes, creating new classes and interfaces, so that the project would be healthy for expansion.

2 - It was set new functions so that it woul be in agree with business requests, for further information check explain.md

3 - It was created unit tests, but they are superficial

Jira link:

No Jira link associated

How:

Following Kotlin best practices, building project initially with Intellij and creating additional pastes in order to set developer identity

Comments:

@WildCLown WildCLown changed the title Gabriel Teixeira Challenge New Payment Flow Challenge Aug 14, 2024
@WildCLown
Copy link
Author

Yeah, I forgot to set the gitignore following kotlin practices

https://github.com/JetBrains/kotlin/blob/master/.gitignore

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