Skip to content

assets: add AScoin provider image - #3151

Open
jamilahmadzai wants to merge 1 commit into
Chain-Love:mainfrom
jamilahmadzai:codex/add-ascoin-provider-image
Open

assets: add AScoin provider image#3151
jamilahmadzai wants to merge 1 commit into
Chain-Love:mainfrom
jamilahmadzai:codex/add-ascoin-provider-image

Conversation

@jamilahmadzai

Copy link
Copy Markdown
Contributor

Summary

  • Add the missing AScoin provider image referenced by references/providers/providers.csv as ascoin.png.
  • Use AScoin's official site favicon as the source asset and normalize it to a square transparent PNG.
  • Leave all CSV data unchanged; this only completes the existing provider asset reference.

Type of change

  • Add data rows
  • Update data rows
  • Remove data rows
  • Schema change
  • Documentation/metadata only

Scope

  • Networks affected: existing networks with AScoin wallet listings; no listing data changed
  • Categories affected: provider assets / wallets
  • Added images: 1
  • Estimated reward: $0.10 USDC for one image, subject to the project's final compensation calculation

Open PR #3132 adds Chainup Cloud, MessinaOne, Mosaia, and Saakuru assets, but does not add references/providers/images/ascoin.png. A live file-path check across 522 open PRs found no open PR touching this AScoin image path.

Links

Validation

  • python3 git-hooks/pre-commit.py - passed
  • Staged diff contains only references/providers/images/ascoin.png
  • Image sanity check - PNG, 120x120, RGBA, transparent background
  • Source URL checks - favicon HTTP 200; App Store listing HTTP 200
  • Live open-PR file-path duplicate check - no open PR touching references/providers/images/ascoin.png

Validation checklist

  • I followed the Style Guide and Column Definitions.
  • I personally opened and verified every new link I'm adding. I can confirm, that all the links I'm adding are valid.
  • If I added new entries - I personally confirmed that the provider I'm adding (modifying) currently supports the adjusted network(s). I've also verified that value in every cell I'm changing is correct according to my best understanding.
  • This PR is not a blind AI-generated submission

Optional

  • Rewards address (for data patching rewards): 0xEA102cf5930eec97b0D39B916F82a46eAeA5f1B7
  • Twitter (X) post link (for +10% of rewards to this PR): N/A

@github-actions

Copy link
Copy Markdown

Summary

Status Count
🔍 Total 0
🔗 Unique 0
✅ Successful 0
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 0
❓ Unknown 0
🚫 Errors 0
⛔ Unsupported 0

Full Github Actions output

@USS-Supervisor USS-Supervisor left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Verdict: APPROVE
Risk: LOW
Summary: This adds only the missing AScoin provider image referenced by the existing provider row, without changing CSV data.
Findings: None. The image is a 120x120 8-bit RGBA PNG with transparent corners, references/providers/providers.csv already points ascoin to ascoin.png, required validation passed, and current-cycle link-check run 32635475003 completed successfully with 0 errors/timeouts/unsupported.
Confidence: HIGH

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