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

Consider removing access to global scope object on container runtime #3484

Closed
vladsud opened this issue Sep 3, 2020 · 2 comments
Closed
Assignees
Labels
api design-required This issue requires design thought status: stale
Milestone

Comments

@vladsud
Copy link
Contributor

vladsud commented Sep 3, 2020

Global state is bad. Instead right state should be passed in by caller to fluid object on creation, and fluid objects should serialize required dependencies (including serializing references to other fluid objects via handles) as part of fluid object state for future usage.

@vladsud vladsud added this to the October 2020 milestone Sep 3, 2020
@vladsud vladsud self-assigned this Sep 3, 2020
@ghost ghost added the triage label Sep 3, 2020
@danielroney danielroney added design-required This issue requires design thought and removed triage labels Sep 3, 2020
@vladsud vladsud modified the milestones: October 2020, Next Oct 1, 2020
@curtisman curtisman added the api label Oct 26, 2020
@ghost ghost added the status: stale label Jun 8, 2021
@ghost
Copy link

ghost commented Jun 8, 2021

This issue has been automatically marked as stale because it has had no activity for 180 days. It will be closed if no further activity occurs within 8 days of this comment. Thank you for your contributions to Fluid Framework!

@ghost ghost added the status: stale label Dec 5, 2021
@ghost
Copy link

ghost commented Dec 5, 2021

This issue has been automatically marked as stale because it has had no activity for 180 days. It will be closed if no further activity occurs within 8 days of this comment. Thank you for your contributions to Fluid Framework!

@ghost ghost closed this as completed Dec 13, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api design-required This issue requires design thought status: stale
Projects
None yet
Development

No branches or pull requests

3 participants