Skip to content

FA-6143 add changelog rferenceid and txhash validation on completion#233

Open
OzBalasFB wants to merge 4 commits intomainfrom
FA-6143-add-changelog-rferenceid-and-txhash-validation-on-completion
Open

FA-6143 add changelog rferenceid and txhash validation on completion#233
OzBalasFB wants to merge 4 commits intomainfrom
FA-6143-add-changelog-rferenceid-and-txhash-validation-on-completion

Conversation

@OzBalasFB
Copy link
Copy Markdown
Collaborator

No description provided.

@OzBalasFB OzBalasFB requested a review from yanivmo March 9, 2026 11:26
@yanivmo
Copy link
Copy Markdown
Collaborator

yanivmo commented Mar 25, 2026

The changelog might just confuse. We do not update it properly. I wouldn't start it now.

Copy link
Copy Markdown
Collaborator

@yanivmo yanivmo left a comment

Choose a reason for hiding this comment

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

You must also generate the unified version.

@OzBalasFB OzBalasFB requested a review from yanivmo March 25, 2026 16:02
Copy link
Copy Markdown
Collaborator

@yanivmo yanivmo left a comment

Choose a reason for hiding this comment

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

  1. Roll back the changelog. Just leave it as it was.
  2. Use my exact wording, including capitalization. The AI generated descriptions use ambiguous terminology and put emphasis on wrong things. Use my proposed description, as is, adjusted to each location. I'd just do it manually.

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