unimal-jp / spear

The spear OSS repository
https://late-cloud-6411.spearly.app
MIT License
10 stars 1 forks source link

Make spear-seo feature available in [item].html generated by cms-tag-routing #165

Closed pandaulait closed 1 year ago

pandaulait commented 1 year ago

Describe the bug The spear-seo feature is not available in [alias].html files with cms-tag-routing.

To Reproduce

  1. Create content with tag fields in Spearly CMS.
  2. Generate [alias].html using cms-tag-routing in spear's appropriate file.

    <!--- index.html --->
    <!DOCTYPE html>
    <html lang="en">
    
    <head>
      <meta charset="UTF-8" />
      <meta http-equiv="X-UA-Compatible" content="IE=edge" />
      <meta name="viewport" content="width=device-width, initial-scale=1.0" />
    </head>
    
    <body>
      <main>
        <ul>
          <li cms-item cms-content-type="article" cms-tag-loop="category" cms-option-limit="1000">
            <h1>{%= article_title %}</h1>
            <p>{%= article_#published_at %}</p>
            <img src="{%= article_image %}" alt="サムネイル画像" />
            <div>{%= article_body %}</div>
          </li>
        </ul>
      </main>
    </body>
    </html>
  3. Enter the following code to set ogp for each article using spear-seo in the head of [alias].html https://github.com/unimal-jp/spear/issues/61
    <spear-seo
        cms-content-type="article"
        title="{%= article_title %} | mySite"
        meta-og:url="{%= #seo_site_url %}/pages/{%= article_#alias %}"
      ></spear-seo>
  4. {%= * %} was not rewritten as shown in the image below スクリーンショット 2023-06-07 15 28 14 (1)

Expected behavior A clear and concise description of what you expected to happen.

Desktop (please complete the following information):


バグの説明 spear-seo機能がcms-tag-routingを用いた[alias].htmlファイルで利用できない。

再現するには

  1. Spearly CMSでタグフィールドのあるコンテンツを作成します。
  2. spearの適当なファイルで、cms-tag-routingを用いた[alias].htmlの生成を行います。

    <!--- index.html --->
    <!DOCTYPE html>
    <html lang="en">
    
    <head>
      <meta charset="UTF-8" />
      <meta http-equiv="X-UA-Compatible" content="IE=edge" />
      <meta name="viewport" content="width=device-width, initial-scale=1.0" />
    </head>
    
    <body>
      <main>
        <ul>
          <li cms-item cms-content-type="article" cms-tag-loop="category" cms-option-limit="1000">
            <h1>{%= article_title %}</h1>
            <p>{%= article_#published_at %}</p>
            <img src="{%= article_image %}" alt="サムネイル画像" />
            <div>{%= article_body %}</div>
          </li>
        </ul>
      </main>
    </body>
    </html>
  3. [alias].htmlのheadにspear-seoを用いて、記事ごとにogpなどを設定するために下記のコードを入力します https://github.com/unimal-jp/spear/issues/61
    <spear-seo
        cms-content-type="article"
        title="{%= article_title %} | mySite"
        meta-og:url="{%= #seo_site_url %}/pages/{%= article_#alias %}"
      ></spear-seo>
  4. 下記画像のように、{%= * %}が書き換えられていなかった スクリーンショット 2023-06-07 15 28 14 (1)

予想される行動

  <spear-seo
    cms-content-type="article"
    title="{%= article_title %} | mySite"
    meta-og:url="{%= #seo_site_url %}/pages/{%= article_#alias %}"
  ></spear-seo>

<title> First Article | mySite</title>
<meta name="og:url" value="https://foobar.com/pages/my-first-article" />

デスクトップ (次の情報を入力してください):

pandaulait commented 1 year ago

spear-seoの使い方を間違えていたため。正確には

<html>
  <head>
    <spear-seo>
    </spear-seo>
  </head>
  <body>
  </body>
</html>

ではなく下記が正しかった

<html>
  <head>
  </head>
  <body>
    <spear-seo>
    </spear-seo>
  </body>
</html>