-
-
Notifications
You must be signed in to change notification settings - Fork 92
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
🔀🎉 nameSchema module as replacement for v3 renameFz #2872
base: dev
Are you sure you want to change the base?
Conversation
temptating = templating? 😄 |
…ldings to dispatchBuildingType / buildingType
Despite my argument with kdev (we agreed to argue about spelling alphabets via PN and dont bother the development with stuff like this). What is the current status of this PR? Looking forward to see it ingame 😊 |
I don't have so much time currently. I tried to implement the template preview, but the Settings Vue Component doesn't seem to be compatible with the composition API currently, at least I was unable to satisfy Typescript, though the project could be build. I have a v3 import feature locally, that enables a button to import existing settings from renameFz when the respective localstorage item is detected. If wanted I can add it to this PR as well. |
Typescript doesn't check within vue files while compiling and building LSSM currently. TypeScript Support will also get better when all stores are migrated to setup stores (and settings store is one of them that needs full rewrite), so I hope that maybe we will even reach a point where LSSM-Settings are fully typed, which allows perfect type checking within setting components. But that is just future thinking ^^ The import button would be very nice, push is welcome :) |
Just to be clear: I am not happy with hacky solutions being necessary but I expect them to be much less necessary in the future (hopefully not too far away) ^^ Try are still allowed as long as not-hacky ones are frustratig |
Typescript will fail with:
|
Hmm, I will have a look at it tomorrow. I think, that's one of the situations where it becomes very hacky 😅🙈 |
Found a hacky way where TypeScript doesn't complain: importFromRenameFz: <Omit<Custom<null>, 'isDisabled' | 'value'>>({
type: 'custom',
component: ImportSettingComponent,
default: null,
properties: {},
disabled() {
return (
window.localStorage.getItem('lssm_LSS_RENAMEFZ_STORAGE') ===
null
);
},
} as unknown), I know how hacky that is, but it works and improving Types of the Settings is already on my TODO :) |
Is there anything we can do to help you get this PR merged? 😃 |
What kind of update does this PR provide? Please check
What is included in your update?
nameSchema
renameFz
moduleAdditional notes