Skip to content
This repository was archived by the owner on May 14, 2025. It is now read-only.

CTR needs to support boot property styles for the tablePrefix property#5857

Merged
onobc merged 2 commits intospring-attic:mainfrom
cppwfs:SCDF-5854
Aug 1, 2024
Merged

CTR needs to support boot property styles for the tablePrefix property#5857
onobc merged 2 commits intospring-attic:mainfrom
cppwfs:SCDF-5854

Conversation

@cppwfs
Copy link
Contributor

@cppwfs cppwfs commented Jul 2, 2024

See #5854

@cppwfs cppwfs requested review from corneil and onobc July 2, 2024 21:45
Copy link
Contributor

@klopfdreh klopfdreh left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

cppwfs added 2 commits July 9, 2024 11:55
…chemas

Also resolved a bug where if a user specified the prefix on CTR for the apps instead of the apps themselves those props would be ignored
@cppwfs cppwfs changed the title CTR needs to support boot property styles for the tablePrefix property [Do Not Merge] CTR needs to support boot property styles for the tablePrefix property Jul 10, 2024
@cppwfs cppwfs changed the title [Do Not Merge] CTR needs to support boot property styles for the tablePrefix property CTR needs to support boot property styles for the tablePrefix property Jul 10, 2024
@cppwfs cppwfs modified the milestones: 2.11.4, 2.11.5 Jul 12, 2024
Copy link
Contributor

@onobc onobc left a comment

Choose a reason for hiding this comment

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

LGTM - thanks @cppwfs

@onobc onobc merged commit 8f6a793 into spring-attic:main Aug 1, 2024
@klopfdreh
Copy link
Contributor

Hey @onobc and @cppwfs - I don't know the workflow, but will this also be merged into main-3?

@cppwfs
Copy link
Contributor Author

cppwfs commented Aug 5, 2024

Hello @klopfdreh Thank you for replying . We are making sure all changes that apply to main-3 will be applied.

@klopfdreh
Copy link
Contributor

Hey @cppwfs - thanks a lot!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants