Properties values should not be limited to String

[zdila]

Property value in MVT can be also number or bool.

[codeart1st]

I get your point. To simplify things, this library currently exports every value in its string representation.

mvt-reader/src/lib.rs

Lines 245 to 268 in c4dfe27

	fn get_string_value(value: vector_tile::tile::Value) -> String {
	if value.string_value.is_some() {
	return value.string_value.unwrap();
	}
	if value.float_value.is_some() {
	return value.float_value.unwrap().to_string();
	}
	if value.double_value.is_some() {
	return value.double_value.unwrap().to_string();
	}
	if value.int_value.is_some() {
	return value.int_value.unwrap().to_string();
	}
	if value.uint_value.is_some() {
	return value.uint_value.unwrap().to_string();
	}
	if value.sint_value.is_some() {
	return value.sint_value.unwrap().to_string();
	}
	if value.bool_value.is_some() {
	return value.bool_value.unwrap().to_string();
	}
	String::new()
	}

[github-actions]

🎉 This issue has been resolved in version 1.8.0-alpha.1 🎉

The release is available on:

• v1.8.0-alpha.1
• npm package (@alpha dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[codeart1st]

Does this suit your needs?

[zdila]

Looks great, thanks! Just please include PartialEq to #[derive(...)] of Value 🙏. And maybe also PartialOrd.

[zdila]

Now it is perfect for me. Thank you! Feel free to close this issue.

PS: version should be 2.0.0 if you follow semantic versioning 😉

[codeart1st]

PS: version should be 2.0.0 if you follow semantic versioning 😉

I appreciate that

[github-actions]

🎉 This issue has been resolved in version 2.0.0 🎉

The release is available on:

• v2.0.0
• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀