-
Notifications
You must be signed in to change notification settings - Fork 50
Closed
Labels
I3Minimal impactMinimal impactS4RoutineRoutineU3RegularRegularconfigConfiguration format update or breaking changeConfiguration format update or breaking changeenhancementImproving existing functionalityImproving existing functionalityneofs-irInner Ring node application issuesInner Ring node application issues
Milestone
Description
Is your feature request related to a problem? Please describe.
I'm always frustrated when we're not grouping our settings clearly and have excessive top level configuration.
Describe the solution you'd like
- Move
without_mainnetintomainnet.disabled, same meaning, same default, keep the old one for a release (compatibility). - Move
governance.disableintomainnet.disable_governance_sync, same behavior, as above. - Move
fee.main_chainintomainnet.extra_fee, same behavior, as above. - Move
contractsintomainnet.contracts, same behavior (just two contracts there), as above.
Describe alternatives you've considered
Keeping things as is, works too.
Additional context
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
I3Minimal impactMinimal impactS4RoutineRoutineU3RegularRegularconfigConfiguration format update or breaking changeConfiguration format update or breaking changeenhancementImproving existing functionalityImproving existing functionalityneofs-irInner Ring node application issuesInner Ring node application issues