algorand / algorand/java-algorand-sdk

Allow TransactionBuilder to accept `null` for `fee`

Open
#523 9 comments 0 reactions 0 assignees View on GitHub
good first issue Team Lamprey
Dominant language
Java
Stars
73
Forks
71
Avg merge
41m
Merged PRs (30d)
2

Description

## Problem

I'd like to rely on suggested params to populate most of the fields but when overriding the `fee` on the transaction with a `flatFee` the builder errors since both `fee` and `flatFee` are set.

If I try to pass `null` to the `fee` method, it only works in the case of `BigInteger` which may be changed at some point.

## Solution

Allow `fee` method in builder to accept `null` for Integer or Long so we can properly unset the fee parameter while still allowing `suggestedParameters` to be passed instead of individual arguments.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.