Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix] 「Download All」ボタンが表示されない #509

Merged
merged 3 commits into from
Jan 20, 2024

Conversation

killinsun
Copy link
Contributor

拡張機能をダウンロードして使ってみたのですが、うまく動かなかったためソースを拝見いたしました。
最新のFantiaの仕様ですと、Observer で取得できる項目に post-btns を親に含むコンテンツがありません。
代わりに、空のpost-show タグ内 にメインコンテンツを埋め込んでいるため条件が通っていなかったようです。

修正の方針として、タグ名でチェックするか、要素を含んでいればOKだったのかの判断がつかなかったため、後者で修正をお送りしております。

@killinsun killinsun changed the title [Fix] ダウンロードボタンが表示されない [Fix] 「Download All」ボタンが表示されない Jan 18, 2024
@mnao305
Copy link
Owner

mnao305 commented Jan 19, 2024

PRありがとうございます!確認します 🙇

@killinsun
Copy link
Contributor Author

ありがとうございます。CI/CDでコケてるようなので、後ほど確認して私の方で対応可能でしたらまた修正上げます🙇‍♂️

@killinsun killinsun closed this Jan 20, 2024
@killinsun killinsun reopened this Jan 20, 2024
@mnao305
Copy link
Owner

mnao305 commented Jan 20, 2024

手元で動作確認して問題なく動作するようになっていました。ありがとうございます!

CI上ではajvが見つからないと怒っているようですが、手元の環境だと特に問題なくビルドできますね…

@killinsun
Copy link
Contributor Author

CIのログ見ると別ブランチでも発生しているようですね。
現在対応されている様子ですので、一旦待ちます 🙏

@mnao305
Copy link
Owner

mnao305 commented Jan 20, 2024

どこかで依存周りが壊れていたのかもです😟

@killinsun
masterに修正を反映したので取り込んで再度pushしていただけますか?

@mnao305 mnao305 merged commit cdff05f into mnao305:master Jan 20, 2024
1 check passed
@mnao305
Copy link
Owner

mnao305 commented Jan 20, 2024

ご対応ありがとうございました!

@killinsun
Copy link
Contributor Author

取り込みありがとうございます!今後もよろしくお願いします😃

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