Skip to content

Update enho#803

Open
huber-nicolas wants to merge 3 commits into
mainfrom
huber-nicolas-patch-1
Open

Update enho#803
huber-nicolas wants to merge 3 commits into
mainfrom
huber-nicolas-patch-1

Conversation

@huber-nicolas

Copy link
Copy Markdown
Contributor

Rename titles of outer_or and inner_or to just "or"
add structure "inner_or" around filter on last "or" level

@hyperspace-insights

Copy link
Copy Markdown
Contributor

Control Panel

Hi, I'm an AI-powered Review Bot that helps you with summarizing and reviewing pull requests.
To interact with me, just use the following actions:

  • 📝 Summarize PR
  • 🔍 Review
  • 🗑️ Delete all bot comments and reviews

@hyperspace-insights hyperspace-insights Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The PR introduces a structural breaking change by wrapping ty_filter in ty_inner_or within the ty_and.or table, which alters the JSON schema of existing or entries. This should be explicitly justified (confirming ENHO format is not yet in production use) or addressed with a version bump — similar to how past PRs handled incompatible v1 changes.

PR Bot Information

Version: 1.26.11

Comment thread file-formats/enho/type/zif_aff_enho_v1.intf.abap
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.

1 participant