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

Move app-tree-utils into repo #9

Merged
merged 1 commit into from
Jul 11, 2022
Merged

Move app-tree-utils into repo #9

merged 1 commit into from
Jul 11, 2022

Conversation

CherryPerry
Copy link
Collaborator

@CherryPerry CherryPerry commented Jul 4, 2022 •
edited
Loading

Move app-tree-utls into this repo.

TODO after merge:

@KovalevAndrey
Copy link
Collaborator

KovalevAndrey commented Jul 4, 2022 •
edited
Loading

I remember the intention was to put all common classes in the common repository, not just customisations.
PermissionsHandler, ActivityStarter, etc.

@zsoltk cc

@CherryPerry CherryPerry mentioned this pull request Jul 8, 2022
Merged
@CherryPerry
Copy link
Collaborator Author

I remember the intention was to put all common classes in the common repository, not just customisations.

I am not ready to do it right now as I am solving a different problem related to repo list. I suggest to do it in a separate PR.

@CherryPerry CherryPerry merged commit a94bb45 into main Jul 11, 2022
@CherryPerry CherryPerry deleted the app-tree-utils branch July 11, 2022 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LachlanMcKee LachlanMcKee LachlanMcKee approved these changes

@KovalevAndrey KovalevAndrey KovalevAndrey approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@CherryPerry @KovalevAndrey @LachlanMcKee