Skip to content
This repository has been archived by the owner on Jan 18, 2022. It is now read-only.

Lay groundwork for serialization overrides #1061

Merged
merged 5 commits into from
Jul 24, 2019

Conversation

jamiebrynes7
Copy link
Contributor

Description

  • Added options for serialization overrides for schema types only.
  • Added this to the GdkToolsConfiguration.json, not exposed in the UI.
  • Overrides are specified by key-value pairs like so: "TargetTypeFqn;SerializationOverrideStaticClassFqn"
  • These are parsed and pushed into the UnityTypeDetails and replaces the content of the Serialize/Deserialize functions for that type.

Tests

Added unit test. Will do an end-to-end test.

Documentation

Changelog. No public docs.

Primary reviewers

If your change will take a long time to review, you can name at most two primary reviewers who are ultimately responsible for reviewing this request. @ mention them.

@improbable-prow-robot improbable-prow-robot added jira/no-ticket Indicates a PR has no corresponding JIRA ticket size/M Denotes a PR that changes 40-149 lines, ignoring generated files. A: tooling Area: Tooling labels Jul 24, 2019
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
Co-Authored-By: Paul Balaji <[email protected]>
@jamiebrynes7 jamiebrynes7 merged commit 93d66e4 into develop Jul 24, 2019
@jamiebrynes7 jamiebrynes7 deleted the feature/serialization-overrides branch July 24, 2019 14:28
jessicafalk pushed a commit that referenced this pull request Nov 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A: tooling Area: Tooling jira/no-ticket Indicates a PR has no corresponding JIRA ticket size/M Denotes a PR that changes 40-149 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants