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.

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

🚚 Moving Table to compat.Table #83

Merged
merged 1 commit into from
Aug 15, 2024
Merged

🚚 Moving Table to compat.Table #83

merged 1 commit into from
Aug 15, 2024

Conversation

sherpal
Copy link
Owner

@sherpal sherpal commented Aug 15, 2024

So that it's more inlined with the new Table from SAP.

So that it's more inlined with the new Table from SAP.
Copy link

pull-request-checklist-buddy bot commented Aug 15, 2024

Demo testing checklist 🎨

  • I made a quick tour on the demo locally and everything seems fine
  • I ran sbt demo/fullLinkJS && cd demo && npm run build and it worked

Copy link

pull-request-checklist-buddy bot commented Aug 15, 2024

Bindings testing checklist 🎨

  • I checked that the changes are made are in sync with the supported SAP UI5 version

@sherpal sherpal merged commit 6985b0a into master Aug 15, 2024
3 checks passed
@sherpal sherpal deleted the move-table-v1 branch August 15, 2024 15:52
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