Skip to content

Conversation

@cs-raj
Copy link
Contributor

@cs-raj cs-raj commented Jul 17, 2025

version bump: 0.2.0 -> 0.3.0

@cs-raj cs-raj requested a review from a team as a code owner July 17, 2025 07:39
@cs-raj cs-raj requested review from nadeem-cs and removed request for a team July 17, 2025 07:39
@nadeem-cs nadeem-cs requested a review from Copilot July 17, 2025 09:05
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds support for nested global field references in CRUD operations and bumps the SDK version to v0.3.0.

  • Introduces GlobalFieldRefs and GlobalFieldReference models
  • Extends ContentModelling to include Description and GlobalFieldRefs
  • Updates version from 0.2.0 to 0.3.0 in project props and user agent
  • Updates CHANGELOG with new feature details

Reviewed Changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
Directory.Build.props Bumped <Version> from 0.2.0 to 0.3.0
Contentstack.Management.Core/Models/GlobalFieldRefs.cs Added GlobalFieldRefs model
Contentstack.Management.Core/Models/Fields/GlobalFieldReference.cs Added GlobalFieldReference field type
Contentstack.Management.Core/Models/ContentModelling.cs Added Description and GlobalFieldRefs properties
Contentstack.Management.Core/ContentstackClient.cs Updated internal Version to 0.3.0
CHANGELOG.md Documented v0.3.0 feature set
Comments suppressed due to low confidence (3)

Contentstack.Management.Core/Models/GlobalFieldRefs.cs:27

  • [nitpick] The JSON property name isChild mixes camelCase with other snake_case properties; consider using is_child for consistency with occurrence_count.
        [JsonProperty(propertyName: "isChild")]

Contentstack.Management.Core/Models/ContentModelling.cs:16

  • New Description property is not covered by existing unit tests; consider adding deserialization tests to verify it and the GlobalFieldRefs list are correctly populated.
        [JsonProperty(propertyName: "description")]

Contentstack.Management.Core/Models/GlobalFieldRefs.cs:10

  • [nitpick] Class name GlobalFieldRefs is plural but represents a single reference; consider renaming to GlobalFieldRef for clarity.
    public class GlobalFieldRefs

@cs-raj cs-raj merged commit d5b673c into development Jul 17, 2025
11 checks passed
@cs-raj cs-raj deleted the feat/DX-3047 branch July 17, 2025 09:21
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.

3 participants