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

Implement TODOs after the reshuffle package merge. #123

Merged
merged 5 commits into from
Sep 30, 2020
Merged

Implement TODOs after the reshuffle package merge. #123

merged 5 commits into from
Sep 30, 2020

Conversation

tinevez
Copy link
Contributor

@tinevez tinevez commented Sep 30, 2020

No description provided.

@tinevez
Rename the Feature.remove() method in invalidate().
639a371
@tinevez
Cache featuresOfTargets lists in GraphFeatureUpdateListeners.
ef70264 
This is called potentially very often, so it can become costly.
@tinevez
Make KeyConfigContexts an interface.
bbdd35c 
I did not move it to the mamut packages, because the constant
it declares are used in the Mastodon API. Also the constants
are related to the views, which are also more generic than the
Mamut app.
@tpietzsch tpietzsch merged commit eb4e225 into master Sep 30, 2020
@tpietzsch
Copy link
Contributor

Thanks!

@tpietzsch tpietzsch deleted the jyt-todos branch September 30, 2020 21:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tpietzsch tpietzsch Awaiting requested review from tpietzsch

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tinevez @tpietzsch