Skip to content

feat(FOUR-32025): LOGIN SCREEN >> Review performance: DB access and queries in login can be cached into redis, images#8923

Open
rodriquelca wants to merge 3 commits into
developfrom
feature/FOUR-32025
Open

feat(FOUR-32025): LOGIN SCREEN >> Review performance: DB access and queries in login can be cached into redis, images#8923
rodriquelca wants to merge 3 commits into
developfrom
feature/FOUR-32025

Conversation

@rodriquelca

@rodriquelca rodriquelca commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Issue & Reproduction Steps

Describe the issue this ticket solves and describe how to reproduce the issue (please attach any fixtures used to reproduce the issue).

Solution

  • Remove full app.css from login/auth pages and replace it with a minimal inline stylesheet (login-base-styles) so the login page no longer downloads ~44 KiB of unused CSS.
  • Load only 2 fonts on the critical path (OpenSans-Regular + OpenSans-ExtraBold) with preload + font-display: swap. Drop Light/Bold/Semibold from first paint.
  • Replace Font Awesome icons with inline SVG (password eye + caps-lock warning) to avoid downloading fa-solid-900.woff2 (~77 KiB).
  • Stabilize logo/favicon cache busting in Setting (?v=updated_at instead of a random token every request).
    Guest-safe ProcessMaker bootstrap on login (EventBus no-op stub + packages list) so package scripts do not throw $emit errors.
performance.mov

How to Test

  1. Open /login (guest, logged out).
  2. Confirm UI: navy background + pattern, slogan text readable, logo, form, Remember me, Forgot password, Login button.
  3. Toggle password eye (show/hide) and Caps Lock warning.
  4. Change language via language selector (still works).
  5. Open /password/reset and confirm password toggles work.
  6. Log in successfully and confirm authenticated app still loads Saved Search scripts (e.g. Tasks/Requests search save button present).

Related Tickets & Packages

Code Review Checklist

  • I have pulled this code locally and tested it on my instance, along with any associated packages.
  • This code adheres to ProcessMaker Coding Guidelines.
  • This code includes a unit test or an E2E test that tests its functionality, or is covered by an existing test.
  • This solution fixes the bug reported in the original ticket.
  • This solution does not alter the expected output of a component in a way that would break existing Processes.
  • This solution does not implement any breaking changes that would invalidate documentation or cause existing Processes to fail.
  • This solution has been tested with enterprise packages that rely on its functionality and does not introduce bugs in those packages.
  • This code does not duplicate functionality that already exists in the framework or in ProcessMaker.
  • This ticket conforms to the PRD associated with this part of ProcessMaker.

ci:deploy

@processmaker-sonarqube

Copy link
Copy Markdown

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@vladyrichter

Copy link
Copy Markdown

QA server K8S was successfully deployed https://ci-cc2673d695.engk8s.processmaker.net

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants