-
Notifications
You must be signed in to change notification settings - Fork 16
GEODE-4063: Update trademarks on website #26
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
GEODE-4063: Update trademarks on website #26
Conversation
This commit addresses a number of issues in order to better align the Apache Geode project website with ASF trademark policy. In particular, it: * Adds trademark symbols where necessary * Replaces the site's introductory text with a single-sentence description of Geode This commit also: * Removes seemingly deprecated code that was no longer rending GitHub repository statistics * Adds a link to the ASF privacy policy, as per ASF policy * Updates the page title Additional details at: https://www.apache.org/foundation/marks/ Signed-off-by: Bryan Behrenshausen <bryan.behrenshausen@sas.com>
|
@nitin-amp: As discussed! Please let me know what you think. cc: @raboof, @JinwooHwang |
Thanks @semioticrobotic! |
raboof
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems reasonable overall
| <p class="description">Build high-speed, data-intensive applications that elastically meet performance requirements at any scale.</p> | ||
| <p class="description">Take advantage of Apache Geode's unique technology that blends advanced techniques for data replication, partitioning and distributed processing.</p> | ||
| <p class="description">Apache Geode provides a database-like consistency model, reliable transaction processing, and a shared-nothing architecture to maintain very low latency performance with high concurrency processing.</p> | ||
| <p class="description">Apache Geode™ is a data management platform that provides real-time, consistent access to data-intensive applications throughout widely distributed cloud architectures.</p> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are we sure we don't want these marketing bullets anymore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would prefer to eliminate them here to avoid the "wall of text" effect that occurs when users load the page. I like the way Apache Airflow does it. Personally, I believe:
- This is a lot of words where fewer could suffice, and
- If we have important points about functionality or benefits, we should include them in the more detailed and thorough explanation in the following site section, which seems to serve that purpose.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That said, I was partial to:
Providing low latency, high concurrency data management solutions since 2002
So you'll see I made that slogan the new page title 😄
Thanks, @nitin-amp! Could you point me to them? I'm afraid I can't see them. (Maybe I need more coffee.) |
If you navigate to files changed, you should be able to see/ respond to those 2 comments. |
1094763 to
ac83042
Compare
nitin-amp
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a few comments in code (one is a duplicate, so you can ignore!)
This commit add a section heading, "Key Features," to introduce the site section devoted to Geode's most noteworthy and useful features. Signed-off-by: Bryan Behrenshausen <bryan.behrenshausen@sas.com>
| /* border-bottom: 1px solid #e5e5e5; */ | ||
| } | ||
| .bf-features { | ||
| color: white; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change is related to the new section header I introduced.
a0b31bf to
a46a9ce
Compare
This commit merges upstream changes to this feature branch, resolving a merge conflict. Signed-off-by: Bryan Behrenshausen <bryan.behrenshausen@sas.com>
a46a9ce to
a827b99
Compare
I've resolved my comment conversations after your responses - LGTM @semioticrobotic! |
|
Thanks @semioticrobotic, I approve the changes. |
This commit addresses a number of issues in order to better align the Apache Geode project website with ASF trademark policy. In particular, it:
This commit also:
Additional details at: https://www.apache.org/foundation/marks/