Skip to content

Conversation

@papeh
Copy link
Contributor

@papeh papeh commented Jan 5, 2026

The initial fix to LT-20393 caused a regression with the same steps as LT-20895. Fix the style export update so that the legacy style import continues to work.

DONE: Update Style exports to be importable (export Structure and Function when not the default value)
TODO: figure out how to import Dictionary Views exported from previous FLEx versions

Also:

  • Modernize the import
  • Update a message about the installation CD
  • Fix incorrect exception messages regarding the import (regrettably, this messages is overwritten in release builds)

Change-Id: I7b6e120246cce9614d1a88a8a42b9145d61d5841


This change is Reviewable

@papeh papeh force-pushed the fix/LT-20895 branch 4 times, most recently from 863cb55 to 42c9bba Compare January 6, 2026 22:47
@papeh papeh changed the title Fix style export for undefined structure Fix style export for undefined structure and function Jan 6, 2026
@github-actions

This comment has been minimized.

papeh added 4 commits January 8, 2026 15:45
The initial fix to LT-20393 caused a regression with the same
steps as LT-20895. Fix the style export update so that the
legacy style import continues to work.

Also:
* Modernize the import
* Update a message about the installation CD
* Fix incorrect exception messages regarding the import

Change-Id: I7b6e120246cce9614d1a88a8a42b9145d61d5841
* Remove some TE-specific code
* Fix the comment on EnsureCompatibleFactoryStyle

Change-Id: I610d98564e14c36eb9f2e9a9f28d62acf59219c5
Change-Id: Ia02602d5162747fe480723a6515b1ffcacbeb3e9
Change-Id: I5254909decfc953fd9fa157b8c768d8efd214000
during Dict View import

TODO: unit test
FUTURE WORK: rewrite the style import to handle importing shared
user styles differently from loading new installed styles

Change-Id: Ia05179990090fe118c4a26081d5654da5917e0de
@github-actions
Copy link

github-actions bot commented Jan 8, 2026

⚠️ Commit Message Format Issues ⚠️
commit 3e28610d5b:
1: T5 Title contains the word 'WIP' (case-insensitive): "WIP to roll back a failed style import"

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.

2 participants