[PR #605] [MERGED] scripts: don't export the default attribute schemas #846

New issue

Closed

opened 2026-02-27 09:09:57 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 09:09:57 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/lldap/lldap/pull/605
Author: @nitnelave
Created: 6/15/2023
Status: ✅ Merged
Merged: 6/15/2023
Merged by: @nitnelave

Base: main ← Head: postgres_script

📝 Commits (1)

3c39f99 scripts: don't export the default attribute schemas

📊 Changes

3 files changed (+28 additions, -19 deletions)

View changed files

📝 .github/workflows/docker-build-static.yml (+14 -12)
📝 docs/database_migration.md (+7 -6)
📝 scripts/sqlite_dump_commands.sh (+7 -1)

📄 Description

This would lead to duplicates in the schema, they are generated when upgrading the DB to version 5.

It also means that if you somehow change the default attributes (make them hidden/readonly/a list) the changes won't be ported over. The edge case should be rare enough, though.

_{🔄 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/lldap/lldap/pull/605 **Author:** [@nitnelave](https://github.com/nitnelave) **Created:** 6/15/2023 **Status:** ✅ Merged **Merged:** 6/15/2023 **Merged by:** [@nitnelave](https://github.com/nitnelave) **Base:** `main` ← **Head:** `postgres_script` --- ### 📝 Commits (1) - [`3c39f99`](https://github.com/lldap/lldap/commit/3c39f99704deba01aef30c6caf6b0de021977173) scripts: don't export the default attribute schemas ### 📊 Changes **3 files changed** (+28 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/docker-build-static.yml` (+14 -12) 📝 `docs/database_migration.md` (+7 -6) 📝 `scripts/sqlite_dump_commands.sh` (+7 -1) </details> ### 📄 Description This would lead to duplicates in the schema, they are generated when upgrading the DB to version 5. It also means that if you somehow change the default attributes (make them hidden/readonly/a list) the changes won't be ported over. The edge case should be rare enough, though. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>