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

Support chrome.storage when available #213

Merged
merged 19 commits into from
Oct 29, 2024
Merged

Support chrome.storage when available #213

merged 19 commits into from
Oct 29, 2024

Conversation

qrtp
Copy link
Collaborator

@qrtp qrtp commented Oct 22, 2024

Refactors the direct usage of localStorage.* to the use of a storage wrapper. Within the wrapper, the implementation will use chrome.storage when available, and fallback to localStorage if needed. This allows the same codebase to be used on a website and within a chrome extension.

Additional changes:

  • Refactor wallet component color palette for readability / flexibility
  • Change owner color scheme to neutral gray vs. heavy blue
  • Improve fund wallet CTA
  • Click anywhere to hide wallet menu

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 22, 2024

/gcbrun

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 22, 2024

/gcbrun

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 22, 2024

/gcbrun

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 23, 2024

/gcbrun

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 23, 2024

/gcbrun

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 23, 2024

/gcbrun

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 23, 2024

/gcbrun

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 24, 2024

/gcbrun

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 24, 2024

/gcbrun

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 25, 2024

/gcbrun

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 25, 2024

/gcbrun

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 27, 2024

/gcbrun

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 28, 2024

/gcbrun

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 28, 2024

/gcbrun

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 28, 2024

/gcbrun

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

@qrtp
Copy link
Collaborator Author

qrtp commented Oct 29, 2024

/gcbrun

Copy link

🎉 All your tests were successful!

We detected changes in the following:

  • UD.me frontend
  • Shared packages

@qrtp qrtp merged commit 7459600 into main Oct 29, 2024
2 checks passed
@qrtp qrtp deleted the qrtp/chrome-storage branch October 29, 2024 14:19
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.

1 participant