Hello, I'm Franklin Bot and I will run some test suites that validate the page speed.
In case there are problems, just click the checkbox below to rerun the respective action.

• Re-run PSI Checks

Hello, I'm the AEM Code Sync Bot and I will run some test suites that validate the page speed.
In case there are problems, just click the checkbox below to rerun the respective action.

• Re-run PSI Checks

a colleague of my team reminded me that if Adobe updates lib-franklin, all changes in lib-franklin.js will be lost, so maybe it is better if in the changes that I made, we can revoke them, but not the ones in scripts.js, WDYT @tomasznetcentric

a colleague of my team reminded me that if Adobe updates lib-franklin, all changes in lib-franklin.js will be lost, so maybe it is better if in the changes that I made, we can revoke them, but not the ones in scripts.js, WDYT @tomasznetcentric

@jonatan-lledo-netcentric but make this refactoring sense outside of the lib-franklin.js ?

a colleague of my team reminded me that if Adobe updates lib-franklin, all changes in lib-franklin.js will be lost, so maybe it is better if in the changes that I made, we can revoke them, but not the ones in scripts.js, WDYT @tomasznetcentric

@jonatan-lledo-netcentric but make this refactoring sense outside of the lib-franklin.js ?

not necessarily, at that moment was a matter of seeing if the code could be improved and I started with the core scripts. but currently, there are enough open issues to work with more important than this one. WDYT?

a colleague of my team reminded me that if Adobe updates lib-franklin, all changes in lib-franklin.js will be lost, so maybe it is better if in the changes that I made, we can revoke them, but not the ones in scripts.js, WDYT @tomasznetcentric

@jonatan-lledo-netcentric but make this refactoring sense outside of the lib-franklin.js ?

not necessarily, at that moment was a matter of seeing if the code could be improved and I started with the core scripts. but currently, there are enough open issues to work with more important than this one. WDYT?

Just to clarify: lib-franklin.js will not be auto updated by Adobe, we always have to pull changes manually from the boilerplate if we want to update. Having that said we should avoid making changes which are not functional to minimise effort if we want to take over changes.