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

Passkey energy v0 #14

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Passkey energy v0 #14

wants to merge 2 commits into from

Conversation

depatchedmode
Copy link

Summary

I've updated the copy, styling and branding to prepare for the Passkey.Energy launch, with links back to the odd.dev website. Related issue is open to map the domain, here: https://github.com/fission-codes/infrastructure/issues/227

Align with our Passkey.Energy stickers in preparation for handing out at DWeb camp and raising awareness about the path to enabling passkeys as an encryption factor.

Test plan (required)

Demonstrate the code is solid. Example: The exact commands you ran and their output, screenshots / videos if the pull request changes UI.

Closing issues

Fixes #

After Merge

  • Does this change invalidate any docs or tutorials? If so ensure the changes needed are either made or recorded
  • Does this change require a release to be made? Is so please create and deploy the release

Copy link
Member

@hugomrdias hugomrdias left a comment

Choose a reason for hiding this comment

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

These changes should be made to the /examples/demo folder thats the one deploy as a fission app. The odd-passkeys-preact is just a starter example doesnt need much branding.

Can you please add a logo svg similar to the previous brand.svg so i can generate PWA icons ?

@depatchedmode
Copy link
Author

@hugomrdias Made the requested changes. Updated logo.svg and logo-borderless.svg with just the standalone Passkey.Energy symbol (w/o the wordmark). You can generate PWA icons from that. Let me know if you need anything else.

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.

2 participants