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

fix: [M3-7151] - Redis trademark changes #9694

Merged
merged 12 commits into from
Sep 18, 2023

Conversation

hmorris3293
Copy link
Contributor

@hmorris3293 hmorris3293 commented Sep 18, 2023

Description 📝

Added Redis Legal verbiage to the app descriptions and any instance of Redis includes the Trademark logo

Preview 📷

image

Single node Cluster
image image

How to test 🧪

  • Go to /linodes/create?type=One-Click
  • Ensure mentions of redis include a trademark
  • Ensure searching for redis still displays the two apps
  • Ensure the drawer still opens and that there is a legal disclaimer in the drawer
  • Redis logo should be replaced by 3 generic stacked slices

@hmorris3293 hmorris3293 requested a review from a team as a code owner September 18, 2023 17:14
@hmorris3293 hmorris3293 requested review from mjac0bs and bnussman-akamai and removed request for a team September 18, 2023 17:14
@hana-akamai hana-akamai self-requested a review September 18, 2023 17:16
@hana-akamai hana-akamai changed the title [Marketplace]Add Redis Trademark info fix: Add Redis trademark info Sep 18, 2023
@hana-akamai hana-akamai changed the base branch from develop to release-v1.102.0 September 18, 2023 18:00
@hana-akamai hana-akamai changed the title fix: Add Redis trademark info fix: Redis trademark changes Sep 18, 2023
jdamore-linode and others added 9 commits September 18, 2023 14:15
…C-specific pricing (linode#9674)

* Move billing and payment tests to their own directory

* Refactor billing activity smoke tests

* Add invoice-related QA data attributes

* Add pagination-related QA data attributes

* Add invoice tests
* feat: [M3-7141] add codeowner file to monorepo root

* feat: [M3-7141] change team name
Copy link
Member

@bnussman-akamai bnussman-akamai left a comment

Choose a reason for hiding this comment

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

Looks good!

Does this have to been in quotes? Feels a bit weird to me

Copy link
Contributor

@mjac0bs mjac0bs left a comment

Choose a reason for hiding this comment

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

Are these fields also in need of the trademark in the Marketplace App for Redis® Sentinel Cluster Setup?
image

Confirmed:
✅ Mentions of Redis® include a trademark
✅ Searching still displays the two apps (both by search bar and by category - Databases)
✅ Drawer still opens and there is a legal disclaimer in the drawer - had a minor question about the quotes around the legal disclaimer, but nothing to hold up the release
✅ Logo should be replaced by 3 generic stacked slices
✅ Guide link redirects as expected

packages/manager/src/features/OneClickApps/oneClickApps.ts Outdated Show resolved Hide resolved
@hana-akamai
Copy link
Contributor

@mjac0bs Holden will look into the Setup inputs asynchronously

Copy link
Contributor

@mjac0bs mjac0bs left a comment

Choose a reason for hiding this comment

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

🚢

@hana-akamai hana-akamai changed the title fix: Redis trademark changes fix: [M3-7151] - Redis trademark changes Sep 18, 2023
@hana-akamai hana-akamai merged commit 15315ce into linode:release-v1.102.0 Sep 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants