Skip to content

Unify App validation across the codebase#317

Open
mirkoCrobu wants to merge 8 commits intomainfrom
apply-app-specs
Open

Unify App validation across the codebase#317
mirkoCrobu wants to merge 8 commits intomainfrom
apply-app-specs

Conversation

@mirkoCrobu
Copy link
Copy Markdown
Contributor

Motivation

closes #464

Change description

Additional Notes

Reviewer checklist

  • PR addresses a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.

@mirkoCrobu mirkoCrobu self-assigned this Mar 27, 2026
@per1234 per1234 added the enhancement New feature or request label Mar 28, 2026
@mirkoCrobu mirkoCrobu marked this pull request as ready for review March 31, 2026 17:03
@mirkoCrobu mirkoCrobu requested review from a team and dido18 April 1, 2026 09:56
Copy link
Copy Markdown
Contributor

@lucarin91 lucarin91 left a comment

Choose a reason for hiding this comment

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

The PR still doesn't unify app validation, you are using two different codes for app.Load an ImportAppFromZip

@mirkoCrobu mirkoCrobu requested a review from lucarin91 April 8, 2026 07:05
@mirkoCrobu mirkoCrobu requested a review from a team April 8, 2026 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants