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

アクションアイコンを取得する機能を廃止する #437

Open
wbonbon opened this issue Dec 14, 2024 · 5 comments
Open

アクションアイコンを取得する機能を廃止する #437

wbonbon opened this issue Dec 14, 2024 · 5 comments
Assignees

Comments

@wbonbon
Copy link
Collaborator

wbonbon commented Dec 14, 2024

ロゴスアクション https://xivapi.com/EurekaMagiaAction/
ロストアクション https://xivapi.com/MYCTemporaryItem/

理由 #435

@wbonbon wbonbon self-assigned this Dec 14, 2024
@wbonbon
Copy link
Collaborator Author

wbonbon commented Dec 14, 2024

アイコンを取得するときIsPlayerActionが0のものを取得してしまうとプレイヤーのものなのかNPCのものなのか分からなくなる
ジョブのアクションアイコンはIsPlayerAction != 0でフィルタする
その結果ロゴスアクションとロストアクションのアイコンがなくなる
代わりにロゴスアクション/ロストアクションを取得して追加する

@wbonbon wbonbon pinned this issue Dec 14, 2024
@wbonbon
Copy link
Collaborator Author

wbonbon commented Dec 14, 2024

旧スタイルのAPIはもうダメっていわれたので
https://beta.xivapi.com/api/1/sheet/MYCTemporaryItem の形式に変更する
他のヤツも…

@wbonbon wbonbon changed the title [todo] スペスペのアクションアイコンからロゴス・ロストアクションを分離する [todo] アクションアイコンを取得する機能を廃止する Dec 15, 2024
@wbonbon
Copy link
Collaborator Author

wbonbon commented Dec 15, 2024

xivapi.comから全てのジョブアクションを取得している機能ですが、
元々実装されている方法ではよろしくないのと
本来実装すべき方法に変更するには私の手に余るためこの機能を削除します
今後アイコンが必要な人は各自用意してもらうことになります

とりあえずUIいじってクリックできないようにしました

@wbonbon wbonbon closed this as completed Dec 15, 2024
@wbonbon wbonbon changed the title [todo] アクションアイコンを取得する機能を廃止する アクションアイコンを取得する機能を廃止する Dec 16, 2024
@Bing-su
Copy link
Contributor

Bing-su commented Dec 16, 2024

古いAPIが使えなくなるまで、この機能をオープンにしておくのはどうでしょうか?まだ必要としている人がいます。

@wbonbon
Copy link
Collaborator Author

wbonbon commented Dec 16, 2024

元々実装されている機能自体がxivapi.comに対して無用な負荷をかけているため使うべきではありません
そして正しい方法に変更するにはコストがかかりすぎます
グローバル版公式サイトのファンキットのページからダウンロードしてください
という方針にしました
ジョブガイドをスクレイピングするpowershellスクリプトを作りましたが、
動作させるための前提条件を整えるのが難しいようです

@wbonbon wbonbon reopened this Dec 16, 2024
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

No branches or pull requests

2 participants