Skip to content

Require data source field mapping and stop leaking documents as titles - #113

Merged
MikeAlhayek merged 2 commits into
mainfrom
ma/fix-datasources
Jul 27, 2026
Merged

Require data source field mapping and stop leaking documents as titles#113
MikeAlhayek merged 2 commits into
mainfrom
ma/fix-datasources

Conversation

@MikeAlhayek

Copy link
Copy Markdown
Member
  • Require key, title, and content field mappings in the MVC and Blazor data source create/edit forms.
  • Fall back to the document key instead of the serialized source document when no title is mapped.
  • Sanitize data source citation titles so serialized documents are never rendered as reference titles.

- Require key, title, and content field mappings in the MVC and Blazor data source create/edit forms.
- Fall back to the document key instead of the serialized source document when no title is mapped.
- Sanitize data source citation titles so serialized documents are never rendered as reference titles.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions

Copy link
Copy Markdown

This pull request has merge conflicts. Please resolve those before requesting a review.

@MikeAlhayek

Copy link
Copy Markdown
Member Author

@copilot resolve the merge conflicts in this pull request

# Conflicts:
#	src/CrestApps.Core.Docs/docs/changelog/v1.0.0.md

Co-authored-by: MikeAlhayek <24724371+MikeAlhayek@users.noreply.github.com>

Copilot AI commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Resolved. The only conflict was in src/CrestApps.Core.Docs/docs/changelog/v1.0.0.md — both branches added a new bullet point at the end of the same list. Both entries are now included in the merge commit 69812ec.

@MikeAlhayek
MikeAlhayek merged commit a7d8400 into main Jul 27, 2026
10 checks passed
@MikeAlhayek
MikeAlhayek deleted the ma/fix-datasources branch July 27, 2026 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants