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

Update dependencies with fixes for SDK changes #235

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lanaklein14
Copy link

Re-opening #232 with some additional fixes which are causing build.bat to fail.

  • Language.Unknown no longer exist in FFXIV_ACT_Plugin.Common Version 2.6.2.0
  • StringHelper is no longer supported in FFXIV_ACT_Plugin.Common Version 2.6.2.0

Also updated the framework version for AddonExample.

@quisquous
Copy link

@ngld Sorry for ping, but would you consider merging this? Are you building against an older version of the ffxiv plugin?

@ngld
Copy link
Owner

ngld commented Apr 12, 2022

I generally only update the references when I need something that's not available in the old version. If I require a too recent version, it can cause ACT to disable OverlayPlugin if a player updates OverlayPlugin before the FFXIV plugin or sometimes for users of the KR or CN versions (since their version number sometimes lags behind).

For example, the current KR version number is 2.1.3.0. If I merge this change, it'll most likely break OverlayPlugin for all KR users without providing any other benefit (as the current build still works with the latest FFXIV plugin).

@quisquous
Copy link

Ok, I see what the issue is on my end. Fetch deps doesn't complain if the ACT plugin is newer, and so I was having the above errors that this is trying to fix.

I think it would still be nice to land the AddOnExample fix (for .net framework) and the other code changes that make it forward-compatible to the new ACT plugin.

quisquous referenced this pull request in OverlayPlugin/OverlayPlugin Aug 19, 2022
Without this, AddonExample throws errors when building.

This is split from #235 to make it easier to land.
quisquous referenced this pull request in OverlayPlugin/OverlayPlugin Aug 19, 2022
Without this, AddonExample throws errors when building.

This is split from #235 to make it easier to land.
quisquous added a commit to OverlayPlugin/OverlayPlugin that referenced this pull request Aug 19, 2022
Remove Language.Unknown and calls to ToProperCase.

Additionally, stop proper casing 6.x versions as the ffxiv plugin has
stopped proper casing names there.  This caused problems in cactbot
where names no longer matched, so doing it here as well for consistency.

Split from ngld#235 to make this easier to land.
This is a copy of ngld#258 applied to this fork.
quisquous added a commit to OverlayPlugin/OverlayPlugin that referenced this pull request Aug 19, 2022
Remove Language.Unknown and calls to ToProperCase.

Additionally, stop proper casing 6.x versions as the ffxiv plugin has
stopped proper casing names there.  This caused problems in cactbot
where names no longer matched, so doing it here as well for consistency.

Split from ngld#235 to make this easier to land.
This is a copy of ngld#258 applied to this fork.
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.

4 participants