Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[ios, macos] Rename featureProperties to featureAttributes. #11748

Merged
merged 4 commits into from
Apr 23, 2018

Conversation

fabian-guerra
Copy link
Contributor

Fixes #11739

@fabian-guerra fabian-guerra added GL JS parity For feature parity with Mapbox GL JS iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS labels Apr 20, 2018
@fabian-guerra fabian-guerra added this to the ios-v4.0.1 milestone Apr 20, 2018
@fabian-guerra fabian-guerra self-assigned this Apr 20, 2018
@fabian-guerra fabian-guerra requested a review from 1ec5 April 20, 2018 20:10
@fabian-guerra fabian-guerra changed the title Fabian feature properties 11739 [ios, macos] Rename featureProperties to featureAttributes. Apr 20, 2018
Copy link
Contributor

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

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

Thanks for making this change! The rename probably deserves a small mention in the iOS and macOS changelogs.

+ (NSExpression *)featureAttributesVariableExpression {
return [NSExpression expressionForVariable:@"featureAttributes"];
}

+ (NSExpression *)featurePropertiesVariableExpression {
return [NSExpression expressionForVariable:@"featureProperties"];
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: return self.featureAttributesVariableExpression so there are fewer code paths.

@1ec5 1ec5 added refactor runtime styling and removed GL JS parity For feature parity with Mapbox GL JS labels Apr 20, 2018
@fabian-guerra fabian-guerra merged commit a45670c into release-boba Apr 23, 2018
@fabian-guerra fabian-guerra deleted the fabian-feature-properties-11739 branch April 23, 2018 13:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
iOS Mapbox Maps SDK for iOS macOS Mapbox Maps SDK for macOS refactor runtime styling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants