[PR #1203] [MERGED] Make bootstrap icons available on dockerized installs #1336

Closed
opened 2026-02-25 21:37:58 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/cypht-org/cypht/pull/1203
Author: @indridieinarsson
Created: 8/28/2024
Status: Merged
Merged: 8/28/2024
Merged by: @Shadow243

Base: masterHead: issue_1174


📝 Commits (1)

  • 58586d1 Make bootstrap icons available on dockerized installs

📊 Changes

1 file changed (+2 additions, -2 deletions)

View changed files

📝 scripts/config_gen.php (+2 -2)

📄 Description

Pullrequest

bootstrap-icons.woff and bootstrap-icons.woff2 are not in the location where the config_gen.php script is expecting them.

This PR fixes that issue, at least on a containerized setup, where install is performed through the docker/Dockerfile script.

Issues

Checklist

  • This seems to have arisen through commit a6ea5b6 which is for easing integration with Tiki. I'm not familiar with that setup, and can't reliably test it. This PR might actually break what was being fixed there, so needs to be scrutinized a bit w.r.t. different install scenarios.

How2Test

  • Check that logos and characters are displayed correctly (see issue #1174) on the main page for all supported install scenarios (standalone, docker, Tiki)

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/cypht-org/cypht/pull/1203 **Author:** [@indridieinarsson](https://github.com/indridieinarsson) **Created:** 8/28/2024 **Status:** ✅ Merged **Merged:** 8/28/2024 **Merged by:** [@Shadow243](https://github.com/Shadow243) **Base:** `master` ← **Head:** `issue_1174` --- ### 📝 Commits (1) - [`58586d1`](https://github.com/cypht-org/cypht/commit/58586d1ec06474ed6fc787f62912b3beed5b8e1f) Make bootstrap icons available on dockerized installs ### 📊 Changes **1 file changed** (+2 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `scripts/config_gen.php` (+2 -2) </details> ### 📄 Description ## Pullrequest bootstrap-icons.woff and bootstrap-icons.woff2 are not in the location where the config_gen.php script is expecting them. This PR fixes that issue, at least on a containerized setup, where install is performed through the docker/Dockerfile script. ### Issues - fixes #1174 ### Checklist - [ ] This seems to have arisen through commit a6ea5b6 which is for easing integration with Tiki. I'm not familiar with that setup, and can't reliably test it. This PR might actually break what was being fixed there, so needs to be scrutinized a bit w.r.t. different install scenarios. ### How2Test - [ ] Check that logos and characters are displayed correctly (see issue #1174) on the main page for all supported install scenarios (standalone, docker, Tiki) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-25 21:37:58 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/cypht#1336
No description provided.