Skip to content

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