Skip to content
This repository was archived by the owner on Jul 7, 2021. It is now read-only.

#235: optinal BankConnection.owners()#355

Merged
proshin-roman merged 2 commits intomasterfrom
#235-optional-bc-owners
Nov 21, 2020
Merged

#235: optinal BankConnection.owners()#355
proshin-roman merged 2 commits intomasterfrom
#235-optional-bc-owners

Conversation

@ccostin93
Copy link
Collaborator

No description provided.

@ccostin93
Copy link
Collaborator Author

@proshin-roman Unfortunately, this is a breaking change. I had to change
Iterable<Owner> BC.owners to Optional<Iterable<Owner>> BC.owners

@ccostin93 ccostin93 self-assigned this Nov 21, 2020
@ccostin93
Copy link
Collaborator Author

@rultor merge

@rultor
Copy link
Collaborator

rultor commented Nov 21, 2020

@rultor merge

@ccostin93 Thanks for your request. @proshin-roman Please confirm this.

@proshin-roman
Copy link
Owner

@rultor confirm

@rultor
Copy link
Collaborator

rultor commented Nov 21, 2020

@rultor confirm

@proshin-roman I'm sorry, I don't understand you :( Check this page and try again please

@proshin-roman proshin-roman merged commit b5b85d4 into master Nov 21, 2020
@proshin-roman proshin-roman deleted the #235-optional-bc-owners branch November 21, 2020 13:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants