|
1 | | -# Modifications to Formbricks |
| 1 | +# GetUp Modifications |
2 | 2 |
|
3 | 3 | This document tracks all modifications made to the original Formbricks codebase for GetUp's deployment as per AGPLv3 §5(a) requirement for modification disclosure. |
4 | 4 |
|
5 | | -**Initial fork:** commit `056e572a3` (main branch, 2025-10-28) |
| 5 | +**Initial fork:** commit `056e572a3` (main branch, 2025-10-29) |
6 | 6 |
|
7 | 7 | **Fork Repository:** https://github.com/GetUp/formbricks |
8 | 8 |
|
9 | | -**Current Base:** Formbricks v4.1.0-rc (synced 2025-10-30) |
| 9 | +**Current Base:** Formbricks v4.1.0 (synced 2025-10-31) |
10 | 10 |
|
11 | | -**Current GetUp Release:** 4.1.0-rc.1-gu-v1 (2025-10-30) |
| 11 | +**Current GetUp Release:** 4.1.0-gu-v1 (2025-10-31) |
| 12 | + |
| 13 | + |
| 14 | +## 2025-10-31 |
| 15 | + |
| 16 | +Based on Formbricks v4.1.0 |
| 17 | + |
| 18 | +**Commits:** |
| 19 | +- 5b06ef1c2 refactor: conditionally render branding container only when needed |
| 20 | +- 67a9ed0ca fix: sticky navigation buttons in surveys for long questions |
| 21 | + |
| 22 | +### Branding Container Optimization |
| 23 | +**Modified by:** GetUp Engineering Team |
| 24 | + |
| 25 | +**Files:** |
| 26 | +- [file path where branding container rendering was changed] |
| 27 | + |
| 28 | +**Changes:** |
| 29 | +- Modified branding container to render conditionally only when branding is enabled |
| 30 | +- Prevents unnecessary DOM elements when no Formbricks branding is present |
| 31 | + |
| 32 | +**Reason:** Removes unnecessary vertical padding |
12 | 33 |
|
13 | 34 | --- |
14 | 35 |
|
15 | | -## GetUp Modifications |
| 36 | +### Survey Navigation Button Fix |
| 37 | +**Modified by:** GetUp Engineering Team |
| 38 | + |
| 39 | +**Files:** |
| 40 | +- [file path(s) where navigation buttons were fixed] |
| 41 | + |
| 42 | +**Changes:** |
| 43 | +- Fixed sticky positioning of navigation buttons in surveys |
| 44 | +- Ensured buttons remain accessible for long question content |
| 45 | +- [specific CSS/layout changes made] |
| 46 | + |
| 47 | +**Reason:** Improve user experience by ensuring navigation buttons are always accessible regardless of question length |
| 48 | + |
| 49 | +--- |
16 | 50 |
|
17 | 51 | ## 2025-10-30 |
18 | 52 |
|
19 | | -### GitHub Workflows Trigger Changes |
20 | | -**Modified by:** GetUp Engineering Team |
| 53 | +Based on Formbricks v4.1.0-rc |
21 | 54 |
|
22 | 55 | **Commits:** |
23 | 56 | - dea3a08fa (ci/change-workflows-for-getup) ci: customise GitHub workflows for GetUp deployment |
24 | 57 |
|
| 58 | +### GitHub Workflows Trigger Changes |
| 59 | +**Modified by:** GetUp Engineering Team |
| 60 | + |
25 | 61 | **Files:** |
26 | 62 | - `.github/workflows/chromatic.yml` (lines 3-8) |
27 | 63 | - `.github/workflows/sonarqube.yml` (lines 2-10) |
@@ -70,6 +106,8 @@ This document tracks all modifications made to the original Formbricks codebase |
70 | 106 |
|
71 | 107 | ## 2025-10-29 |
72 | 108 |
|
| 109 | +Initial fork from commit `056e572a3` |
| 110 | + |
73 | 111 | **Commits:** |
74 | 112 | - 9fb352eeb (origin/feature/customise-branding, feature/customise-branding) docs: document modifications for AGPLv3 compliance |
75 | 113 | - f3608154e feat: add environment variable to disable Formbricks branding |
|
0 commit comments