Skip to content

fix: InboundService fixes to match backend logic#192

Merged
LuisReyes98 merged 1 commit into
mainfrom
development
May 12, 2026
Merged

fix: InboundService fixes to match backend logic#192
LuisReyes98 merged 1 commit into
mainfrom
development

Conversation

@LuisReyes98
Copy link
Copy Markdown
Contributor

This pull request introduces several improvements and fixes related to the handling of inbound structures, particularly making the position field nullable to support services without a position structure. It also adds new converters for safer deserialization of dynamic maps and updates Flutter and DevTools configuration files. Below are the most important changes:

Inbound Structure Nullability and CopyWith Improvements

  • Made the position field nullable (InboundPositionStructure? and InboundPositionStructureInput?) in InboundStructure and InboundStructureInput, including all relevant methods, constructors, and copyWith implementations, to support services that do not require a position structure. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17] [18] [19] [20] [21]

Deserialization and Converter Enhancements

  • Added DynamicMapConverter and DynamicMapConverterNullable for safe deserialization of Map<dynamic, dynamic> to Map<String, dynamic>, and applied these to InboundService.credentials and InboundServiceInput.credentials to prevent failures when credentials are provided as empty map literals.

Flutter and Tooling Configuration

  • Updated the Flutter version in .fvmrc from 3.41.8 to 3.41.9.
  • Added configuration to devtools_options.yaml to disable drift and shared_preferences extensions by default.

Documentation

  • Updated CHANGELOG.md to document the above changes as part of version 3.8.12.

@github-actions
Copy link
Copy Markdown

📋 Changelog Summary

This PR includes the following changes:

🐛 Bug Fixes


🤖 Auto-generated by changelog workflow

@LuisReyes98 LuisReyes98 merged commit b1d8f7c into main May 12, 2026
1 of 2 checks passed
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.

1 participant