Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Format address as property of Identity #285

Merged
merged 4 commits into from
Dec 5, 2024

Conversation

britsta
Copy link
Contributor

@britsta britsta commented Dec 5, 2024

The address is often a property of an enmeshed Identity and should be formatted accordingly in that cases. In most cases it should at least be written in lowercase (except for sentence beginnings or titles). If it is not the property address of an Identity that is referred to, but the concept of the enmeshed Address, it is a proper name and should still be capitalized and written unformatted.

@britsta britsta added the enhancement New feature or request label Dec 5, 2024
@britsta britsta marked this pull request as ready for review December 5, 2024 14:27
@jkoenig134
Copy link
Member

Hm, I don't like the idea of talking about the Identities Address as only a property b/c its more. For places we really talking about the property (e.g. for copy pasting it to a payload) I'm totally with you, it's just the generalization here that bothers me.

@britsta
Copy link
Contributor Author

britsta commented Dec 5, 2024

Hm, I don't like the idea of talking about the Identities Address as only a property b/c its more. For places we really talking about the property (e.g. for copy pasting it to a payload) I'm totally with you, it's just the generalization here that bothers me.

I understand exactly what you mean (would have been the alternative variant I would have offered to merge) and adjust the passages accordingly.

@britsta britsta merged commit 0e68374 into main Dec 5, 2024
5 checks passed
@britsta britsta deleted the refactor/format-address-as-property-of-identity branch December 5, 2024 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants