Skip to content

Comments

Changes for taro-sitemap issue 23#24

Open
akawallie wants to merge 3 commits intomainfrom
tsmap-issue-23
Open

Changes for taro-sitemap issue 23#24
akawallie wants to merge 3 commits intomainfrom
tsmap-issue-23

Conversation

@akawallie
Copy link
Contributor

  • Attachment サイトマップの自動フラッシュを有効化(前回忘れていました)
  • build_url() にクエリ対応を追加
  • AbstractSitemapProvider.php 以外のすべてのファイルから get_xslt_url() を削除
  • get_xslt_url() を動的にし、クエリ対応を追加
  • 動的なサイトマップURLを生成するために QueryArgsHelper.phpsitemap_url() を作成

@akawallie akawallie linked an issue Aug 27, 2025 that may be closed by this pull request
@akawallie akawallie requested a review from waviaei August 27, 2025 06:01
@akawallie
Copy link
Contributor Author

akawallie commented Aug 28, 2025

@waviaei

もう一つの可能性のある問題に気付きました:#25

@waviaei
Copy link
Member

waviaei commented Sep 9, 2025

@TSMarkWalters ありがとうございます。
#25 の方に @fumikito さんからレビューが返ってきていて、変更ファイルが被っている?のと、あちらの方が複雑なので、まずは #25 の方を優先でお願いします。

Copy link
Member

@waviaei waviaei left a comment

Choose a reason for hiding this comment

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

@TSMarkWalters Sorry, I am rather confused with this PR's relation with #26

@waviaei waviaei requested a review from fumikito October 2, 2025 09:49
@fumikito
Copy link
Contributor

fumikito commented Oct 3, 2025

レビュー状況

  • Attachment サイトマップの自動フラッシュを有効化(前回忘れていました)
  • build_url() にクエリ対応を追加
  • AbstractSitemapProvider.php 以外のすべてのファイルから get_xslt_url() を削除
  • get_xslt_url() を動的にし、クエリ対応を追加
  • 動的なサイトマップURLを生成するために QueryArgsHelper.php に sitemap_url() を作成

Copy link
Contributor

@fumikito fumikito left a comment

Choose a reason for hiding this comment

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

一部レビューしました。残りはあとで。

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.

パーマリンク設定が「Plain」だと 404 になる

3 participants