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

chore: Update Pillar Wallet [Fixes #14360] #14543

Merged
merged 2 commits into from
Jan 8, 2025

Conversation

arddluma
Copy link
Contributor

Description

Updated Pillar Wallet logo and features

Related Issue

#14360

Copy link

netlify bot commented Dec 23, 2024

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit cf8646f
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/6769891c88875000084a8ef2
😎 Deploy Preview https://deploy-preview-14543--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 52 (🔴 down 4 from production)
Accessibility: 92 (no change from production)
Best Practices: 89 (🔴 down 9 from production)
SEO: 98 (no change from production)
PWA: 59 (🟢 up 30 from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@alexandradev alexandradev mentioned this pull request Dec 23, 2024
2 tasks
twBackgroundColor: "bg-[#7501D9]",
twGradiantBrandColor: "from-[#7501D9]",
twBackgroundColor: "bg-[#6A12D0]",
twGradiantBrandColor: "from-[#6A12D0]",
url: "https://www.pillar.fi/",
active_development_team: true,
languages_supported: ["en", "bs", "zh"],
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
languages_supported: ["en", "bs", "zh"],
languages_supported: ["en"],

Copy link
Contributor

Choose a reason for hiding this comment

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

based on the #14360 I see only english support, @arddluma and @alexandradev can you confirm and modify this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hello there,
Please check this screenshot from pillar wallet
IMG_1882

Copy link
Contributor

@konopkja konopkja Dec 23, 2024

Choose a reason for hiding this comment

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

i see, so the issue was filled out incorrectly?

Copy link
Contributor Author

@arddluma arddluma Dec 23, 2024

Choose a reason for hiding this comment

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

yes, apologies for that!

Copy link
Contributor

@konopkja konopkja Dec 23, 2024

Choose a reason for hiding this comment

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

ok no worries, i see 4 languages on the screenshot while only 3 in the metadata, could you pls update it with the missing lang code?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done sir, the missing language is Amharic. I have just added it, do you mind checking it please?

Copy link
Contributor

@konopkja konopkja left a comment

Choose a reason for hiding this comment

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

lgtm, ty

@arddluma
Copy link
Contributor Author

Hey @konopkja , I see that merging is blocked ?
Is there anything required from my side to get this merged ?
Thanks!

@konopkja
Copy link
Contributor

Hey @konopkja , I see that merging is blocked ? Is there anything required from my side to get this merged ? Thanks!

ofc only the core team can merge, nothing else from your side at this point i think. One of our devs will review and merge

Copy link

gitguardian bot commented Dec 30, 2024

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

Since your pull request originates from a forked repository, GitGuardian is not able to associate the secrets uncovered with secret incidents on your GitGuardian dashboard.
Skipping this check run and merging your pull request will create secret incidents on your GitGuardian dashboard.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
- - Generic High Entropy Secret 602d37b src/data/community-events.json View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider

If this detection is a false positive, please contact Oleh Vasylenko (@Aldekein) to fix it in the GitGuardian dashboard.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@corwintines corwintines self-assigned this Jan 2, 2025
@corwintines corwintines merged commit ba4119e into ethereum:dev Jan 8, 2025
6 checks passed
This was referenced Jan 8, 2025
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.

3 participants