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.

Sign up for GitHub

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

Jump to bottom

[Feature] Add an unregister method for AttributeAccessOverride #28072

Closed
hicklin opened this issue Jul 19, 2023 · 0 comments · Fixed by #28075
Closed

[Feature] Add an unregister method for AttributeAccessOverride #28072

hicklin opened this issue Jul 19, 2023 · 0 comments · Fixed by #28075
Assignees
@bzbarsky-apple
Labels
feature request feature work needs triage

Comments

@hicklin
Copy link
Contributor

hicklin commented Jul 19, 2023

Feature description

Add a method to unregister an AttributeAccessOverride like the InteractionModelEngine's UnregisterCommandHandler method for the CommandHandler. This will make it possible for objects that override the attribute accessors to be gracefully destroyed during the application lifetime.

Platform

all

Platform Version(s)

No response

Anything else?

No response
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Jul 19, 2023
@bzbarsky-apple
Add a way to unregister AttributeAccessInterface instances.
11cecc8 
Fixes project-chip#28072
@bzbarsky-apple bzbarsky-apple mentioned this issue Jul 19, 2023
Merged
@mergify mergify bot closed this as completed in #28075 Jul 19, 2023
mergify bot pushed a commit that referenced this issue Jul 19, 2023
@bzbarsky-apple
Add a way to unregister AttributeAccessInterface instances. (#28075)
300881a 
Fixes #28072
erwinpan1 pushed a commit to erwinpan1/connectedhomeip that referenced this issue Jul 21, 2023
@bzbarsky-apple @erwinpan1
Add a way to unregister AttributeAccessInterface instances. (project-…
987d5ae 
…chip#28075)

Fixes project-chip#28072
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bzbarsky-apple bzbarsky-apple

Labels
feature request feature work needs triage
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add a way to unregister AttributeAccessInterface instances. bzbarsky-apple/connectedhomeip
2 participants
@hicklin @bzbarsky-apple