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.

Sign up for GitHub

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

Jump to bottom

Tiny fixes #331

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

Tiny fixes #331

wants to merge 4 commits into from
+158 −47

Conversation

alovak
Copy link
Contributor

@alovak alovak commented Oct 25, 2024

@alovak
do not skip fields in composite if keepzero is set
938552f
@alovak
fix tag parsing
ec4d932
@alovak
allow to use field name and tag options together
3f9cf01 
in case when you don't want to duplicate the field name in the
iso8583 tag, you can just skip it and set the rest of the options like
this:

```go
F2 *field.String `iso8583:",keepzero"`
```

then the field id will be derived from the field name and keepzero
option will be used as well
@alovak
add test for resetting field values
aab1091
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@alovak