Skip to content

Feat/21 app grouping#33

Merged
sugan0tech merged 6 commits into
mainfrom
feat/21-app-grouping
Aug 25, 2025
Merged

Feat/21 app grouping#33
sugan0tech merged 6 commits into
mainfrom
feat/21-app-grouping

Conversation

@venkatsp17

@venkatsp17 venkatsp17 commented Aug 19, 2025

Copy link
Copy Markdown
Contributor

Issue #21

@ssanumand

ssanumand commented Aug 19, 2025

Copy link
Copy Markdown
Member

Looks good to me, please add a reviewer to the PR so that they can review and approve the changes. Once approved, you can merge the branch.

@venkatsp17 venkatsp17 self-assigned this Aug 19, 2025
@venkatsp17 venkatsp17 requested a review from sugan0tech August 19, 2025 18:43
Comment thread internal/models/apps.go Outdated
Comment thread internal/api/handlers/apps.go Outdated
Comment thread internal/api/handlers/apps.go Outdated
Comment thread internal/api/handlers/apps.go Outdated
Comment thread internal/api/handlers/apps.go
@sugan0tech

sugan0tech commented Aug 21, 2025

Copy link
Copy Markdown
Member

@venkatsp17 as with the liting pipeline, it appears to be few functions are unused? Will that be covered in following commits?

@sugan0tech sugan0tech left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, @OpenMigrate/core expecting least one of yours review here.

@sugan0tech sugan0tech merged commit a821fda into main Aug 25, 2025
2 checks passed
@sugan0tech sugan0tech deleted the feat/21-app-grouping branch August 25, 2025 11:29
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.

3 participants