Skip to content

[FIX] generateFlexBundle: Improve flexBundle flagging - #1233

Merged
matz3 merged 1 commit into
SAP:v4from
Lonwyr:fixFlexBundle
Sep 10, 2026
Merged

matz3 merged 1 commit into
SAP:v4from
Lonwyr:fixFlexBundle

Conversation

@Lonwyr

@Lonwyr Lonwyr commented Aug 27, 2026

Copy link
Copy Markdown
Contributor
  • the flag should only be used for the flexibilityBundle
  • to improve the client performance, it now also only sets the flag in case of a mandatory early loading of the bundle. Otherwise, the file can be loaded from the -preload

Thank you for your contribution! 🙌

To get it merged faster, kindly review the checklist below:

Pull Request Checklist

Comment thread lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread lib/tasks/bundlers/generateFlexChangesBundle.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated

@Lonwyr Lonwyr left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

adjusted the property, the JSDoc and all mentioned references. I kept the original comment open in case you have feedback on the current naming proposal

Comment thread lib/tasks/bundlers/generateFlexChangesBundle.js Outdated
Comment thread lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
Comment thread test/lib/processors/bundlers/flexChangesBundler.js Outdated
* the flag should only be used for the flexibilityBundle
* to improve the client performance, it now also only sets the flag in case of a mandatory early loading of the bundle. Otherwise, the file can be loaded from the -preload
@matz3 matz3 changed the title [FIX] generateFlexBundle correct & improve flexBundle flagging [FIX] generateFlexBundle: Improve flexBundle flagging Sep 10, 2026
@matz3
matz3 dismissed d3xter666’s stale review September 10, 2026 13:53

comments were addressed

@matz3
matz3 merged commit c363ec7 into SAP:v4 Sep 10, 2026
19 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