From d54efe78f72c627349c2d07fd692767589e2be4d Mon Sep 17 00:00:00 2001 From: Miguel Serrano Date: Fri, 10 Oct 2025 10:31:40 +0200 Subject: [PATCH] [CE/SP] Fix default footer (#28947) * [CE/SP] Fix default footer Fixes the default `right_footer`. Only 'text' should be defined when it contains HTML code. * Remove font-awesome github icon * Added acceptance test GitOrigin-RevId: 1a1ba04673f35affcec4bb17bc7cd860d589b13f --- server-ce/test/customization.spec.ts | 48 +++++++++++++++--------- services/web/config/settings.defaults.js | 3 +- 2 files changed, 32 insertions(+), 19 deletions(-) diff --git a/server-ce/test/customization.spec.ts b/server-ce/test/customization.spec.ts index 03c9bc3a5a..feabe0cc9e 100644 --- a/server-ce/test/customization.spec.ts +++ b/server-ce/test/customization.spec.ts @@ -2,25 +2,39 @@ import { isExcludedBySharding, startWith } from './helpers/config' describe('Customization', () => { if (isExcludedBySharding('CE_CUSTOM_1')) return - startWith({ - vars: { - OVERLEAF_APP_NAME: 'CUSTOM APP NAME', - OVERLEAF_LEFT_FOOTER: JSON.stringify([{ text: 'CUSTOM LEFT FOOTER' }]), - OVERLEAF_RIGHT_FOOTER: JSON.stringify([{ text: 'CUSTOM RIGHT FOOTER' }]), - }, + + describe('default settings', () => { + startWith({}) + + it('should display the default right footer', () => { + cy.visit('/') + cy.get('footer').findByRole('link', { name: 'Fork on GitHub!' }) + }) }) - it('should display custom name', () => { - cy.visit('/') - cy.get('nav').findByText('CUSTOM APP NAME') - }) + describe('custom settings', () => { + startWith({ + vars: { + OVERLEAF_APP_NAME: 'CUSTOM APP NAME', + OVERLEAF_LEFT_FOOTER: JSON.stringify([{ text: 'CUSTOM LEFT FOOTER' }]), + OVERLEAF_RIGHT_FOOTER: JSON.stringify([ + { text: 'CUSTOM RIGHT FOOTER' }, + ]), + }, + }) - it('should display custom left footer', () => { - cy.visit('/') - cy.get('footer').findByText('CUSTOM LEFT FOOTER') - }) - it('should display custom right footer', () => { - cy.visit('/') - cy.get('footer').findByText('CUSTOM RIGHT FOOTER') + it('should display custom name', () => { + cy.visit('/') + cy.get('nav').findByText('CUSTOM APP NAME') + }) + + it('should display custom left footer', () => { + cy.visit('/') + cy.get('footer').findByText('CUSTOM LEFT FOOTER') + }) + it('should display custom right footer', () => { + cy.visit('/') + cy.get('footer').findByText('CUSTOM RIGHT FOOTER') + }) }) }) diff --git a/services/web/config/settings.defaults.js b/services/web/config/settings.defaults.js index 8d0913f752..99abdfb431 100644 --- a/services/web/config/settings.defaults.js +++ b/services/web/config/settings.defaults.js @@ -778,8 +778,7 @@ module.exports = { right_footer: [ { - text: " Fork on GitHub!", - url: 'https://github.com/overleaf/overleaf', + text: 'Fork on GitHub!', }, ],