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

add use cases for ContextResource class #38

Closed
wants to merge 1 commit into from
Closed

add use cases for ContextResource class #38

wants to merge 1 commit into from

Conversation

Denis-Frunza
Copy link
Contributor

Sequence diagram included to depict interactions between classes.Also included Dict into documentation

@Denis-Frunza
Copy link
Contributor Author

@lesnik512 I decided to start with Sequence diagram. You may have suggestions what ca be done to improve.

@lesnik512
Copy link
Member

@Denis-Frunza thank you) sequence diagram seems too technical. I'm not sure that it would be helpful for library users

@Denis-Frunza
Copy link
Contributor Author

@Denis-Frunza thank you) sequence diagram seems too technical. I'm not sure that it would be helpful for library users

Probably you're right, I'll add use cases with code example.It should work

@Denis-Frunza Denis-Frunza changed the title Sequnce diagram added for DIContainer, ContextResource, and Resource add use cases for ContextResource class Jun 21, 2024
@Denis-Frunza Denis-Frunza marked this pull request as ready for review June 21, 2024 07:55
@Denis-Frunza
Copy link
Contributor Author

Could you please review my pull request when you have a chance? Thank you.

docs/providers/resource.md Show resolved Hide resolved
docs/providers/resource.md Show resolved Hide resolved
docs/providers/resource.md Show resolved Hide resolved
@lesnik512
Copy link
Member

@Denis-Frunza Thank you for efforts. I'll write this doc myself

@lesnik512 lesnik512 closed this Jun 21, 2024
@Denis-Frunza
Copy link
Contributor Author

@Denis-Frunza Thank you for efforts. I'll write this doc myself

as you wish

@Denis-Frunza Denis-Frunza deleted the 15-docs-context-resources branch June 22, 2024 18:05
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.

2 participants