Skip to content

Conversation

@QuickStick123
Copy link
Contributor

@QuickStick123 QuickStick123 commented Aug 21, 2022

Fixes #4971 #5006

This includes the suggest/started changes completed in #2536

3.18.0: The following Unique Items will now be found with both Shaper- and Elder-Influence: Disintegrator, Indigon, Voidfletcher, Voidforge, The Eternity Shroud, and Replica Eternity Shroud. This change does not apply to existing versions of these Unique Items.

I updated all the items but have not added legacy variants for these as would need to add support.

There is a bug which is much more noticable with multibase/level requirement uniques where they don't update.

["^(%d+)"] = "BASE", was added to get # life on block working as it no longer has any + sign.
It is a very broad form which may cause issues.

@QuickStick123 QuickStick123 added the bug: accuracy Wording differences label Aug 21, 2022
@QuickStick123 QuickStick123 changed the title Update wording difference post launch and other changes. Update Unqiue wording difference post launch and other changes. Aug 21, 2022
@QuickStick123 QuickStick123 changed the title Update Unqiue wording difference post launch and other changes. Update Unqiue wording differences post launch and other changes. Aug 21, 2022
@QuickStick123 QuickStick123 marked this pull request as ready for review August 27, 2022 11:21
@QuickStick123 QuickStick123 force-pushed the update-wording-difference-post-launch branch from 486ac2d to d3a645a Compare August 29, 2022 07:13
@QuickStick123 QuickStick123 force-pushed the update-wording-difference-post-launch branch from e5a8716 to 44c2656 Compare September 11, 2022 12:30
Share Endurance Charges with nearby party members
{variant:5}Your nearby party members' maximum Endurance Charges are equal to yours.
{variant:1,2,3,4}Share Endurance Charges with nearby party members
{variant:5}Your nearby party members maximum Endurance Charges is equal to yours
Copy link
Member

Choose a reason for hiding this comment

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

Grammar like this bothers me, but it matches in-game, so c'est la vie

Copy link
Member

@Wires77 Wires77 left a comment

Choose a reason for hiding this comment

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

I've let perfect be the enemy of done on this one for too long, and what this encompasses looks good. Maybe someday I'll be able to merge in my automatic script, heh.

@Wires77 Wires77 merged commit f9b50e2 into PathOfBuildingCommunity:dev Dec 9, 2022
@QuickStick123
Copy link
Contributor Author

QuickStick123 commented Dec 9, 2022

I have reviewed this myself at least 3 times so I am fairly confident it is fine.

@Wires77 Wires77 changed the title Update Unqiue wording differences post launch and other changes. Update Unqiue wording differences Dec 9, 2022
@Wires77 Wires77 changed the title Update Unqiue wording differences Update Unique wording differences Dec 9, 2022
@Wires77
Copy link
Member

Wires77 commented Dec 9, 2022

I have reviewed this myself at least 3 times so I am fairly confident it is fine.

Same, honestly. There wasn't much new since I reviewed it 3 months ago

@QuickStick123 QuickStick123 deleted the update-wording-difference-post-launch branch December 14, 2022 03:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug: accuracy Wording differences

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Viridian Grand Spectrum Value is Incorrect

3 participants