Skip to content

Add xmlns:core to XML example in README#253

Merged
flovogt merged 1 commit intoUI5:mainfrom
anitamalina:patch-3
Apr 22, 2026
Merged

Add xmlns:core to XML example in README#253
flovogt merged 1 commit intoUI5:mainfrom
anitamalina:patch-3

Conversation

@anitamalina
Copy link
Copy Markdown
Contributor

This pull request makes a small update to the XML view file by adding the sap.ui.core namespace as core. This change ensures that core UI5 controls and features can be referenced in the view.

  • Added the xmlns:core="sap.ui.core" namespace declaration to the XML view to enable usage of core UI5 controls.

Copilot AI review requested due to automatic review settings April 20, 2026 12:05
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds the missing sap.ui.core XML namespace prefix (core) to the Step 34 walkthrough README’s InvoiceList.view.xml example so the snippet can validly use core UI5 features (notably the existing core:require usage later in the example).

Changes:

  • Add xmlns:core="sap.ui.core" to the <mvc:View> XML snippet in Step 34 documentation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@flovogt
Copy link
Copy Markdown
Member

flovogt commented Apr 22, 2026

Good catch! Thanks a lot @anitamalina

@flovogt flovogt merged commit df213ca into UI5:main Apr 22, 2026
7 of 8 checks passed
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