Skip to content

feat: add heancms template - #824

Open
splat58 wants to merge 6 commits into
Aidoku-Community:mainfrom
splat58:heancms
Open

splat58 wants to merge 6 commits into
Aidoku-Community:mainfrom
splat58:heancms

Conversation

@splat58

@splat58 splat58 commented Oct 17, 2026

Copy link
Copy Markdown
Contributor

@kkantan kkantan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you may as well add luascans and omegascans to this pr as well to show that the template works. or if you wanted to keep things separate, you could have stacked prs.

Comment thread templates/heancms/src/imp.rs Outdated
Comment thread templates/heancms/src/imp.rs Outdated
Comment thread templates/heancms/src/models.rs Outdated
@splat58

splat58 commented Oct 19, 2026

Copy link
Copy Markdown
Contributor Author

Resolves #609

@splat58
splat58 requested a review from kkantan October 19, 2026 17:30
@kkantan kkantan linked an issue Oct 19, 2026 that may be closed by this pull request

@kkantan kkantan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it possible to support tag search on tap? since genres are collected as a dynamic filter, it's not automatically supported, so you would need to either create a script to statically generate a filters.json, or add supportsTagSearch in the source.json config and handle the "genre" filter in search with only the name of the tag.

Comment thread templates/heancms/src/imp.rs
@splat58
splat58 requested a review from kkantan October 20, 2026 01:25
.description
.and_then(|d| Html::parse_fragment(d).ok())
.and_then(|html| html.select_first("body")?.text_with_newlines())
.map(|d| d.trim().into()),

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's possible for this to be an empty string, in which case it should be returned as None instead.

Suggested change
.map(|d| d.trim().into()),
.map(|d| d.trim().into())
.filter(|d| !d.is_empty()),

Comment thread templates/heancms/src/imp.rs
qs.push(&id, sort);
qs.push("order", Some(if ascending { "asc" } else { "desc" }));
}
FilterValue::MultiSelect { id, included, .. } => {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you don't set can_exclude to false on the genres filter despite not supporting excluded values.

This branch has not been deployed

No deployments
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.

luacomic.org

2 participants