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

Rename to Oasis ROSE Wallet, except in sidebar #1879

Merged
merged 1 commit into from
Apr 10, 2024
Merged

Conversation

lukaw3d
Copy link
Member

@lukaw3d lukaw3d commented Apr 9, 2024

No description provided.

@lukaw3d lukaw3d requested review from buberdds and tjanez April 9, 2024 13:42
@lukaw3d lukaw3d changed the title Prefix ROSE wallet with Oasis when it doesn't have a logo in front Prefix ROSE Wallet with "Oasis" when it doesn't have a logo in front Apr 9, 2024
Copy link

github-actions bot commented Apr 9, 2024

Deployed to Cloudflare Pages

Latest commit: b91a81de9ed7d786e8ae731b1471a806b4461ff8
Status:✅ Deploy successful!
Preview URL: https://e35e7280.oasis-wallet.pages.dev

This comment was marked as resolved.

@@ -0,0 +1 @@
Rename to ROSE Wallet
Copy link
Contributor

Choose a reason for hiding this comment

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

  • Rename to Oasis ROSE Wallet
  • why it's breaking? we do not plan to bump major version with this update, right?

Copy link
Member Author

Choose a reason for hiding this comment

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

Im not sure. Process then?

Copy link
Member

Choose a reason for hiding this comment

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

I would classify it as a feature and not breaking.

We will bump major version when we release extension and web from the same codebase.

@buberdds
Copy link
Contributor

buberdds commented Apr 9, 2024

We should take into account meta tags like title or og:title and use full product name there.

@lukaw3d
Copy link
Member Author

lukaw3d commented Apr 9, 2024

We should take into account meta tags like title or og:title and use full product name there.

aw, i liked the tab name:
image

currently google would look like this:
image

currently slack would look like this:
image

@tjanez should we change these to Oasis ROSE Wallet?

Copy link
Contributor

@buberdds buberdds left a comment

Choose a reason for hiding this comment

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

from code perspective it looks good. Product wise I would wait for Tadej.

@tjanez
Copy link
Member

tjanez commented Apr 10, 2024

@tjanez should we change these to Oasis ROSE Wallet?

Yes, change these to Oasis ROSE Wallet.

@lukaw3d lukaw3d changed the title Prefix ROSE Wallet with "Oasis" when it doesn't have a logo in front Rename to Oasis ROSE Wallet, except in sidebar Apr 10, 2024
@lukaw3d lukaw3d merged commit a38ba48 into master Apr 10, 2024
13 checks passed
@lukaw3d lukaw3d deleted the lw/prefix-rose-wallet branch April 10, 2024 16:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants