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

SEP-0042 Stellar Asset Lists Standard #1440

Merged
merged 17 commits into from
Mar 27, 2024

Conversation

orbitlens
Copy link
Contributor

No description provided.

silence48 and others added 11 commits October 18, 2023 21:29
Fixing errors in markdown
…on standard Stellar terminology

- Add `feedback`, `description`, `assets.domain` fields
- Add "Security Concerns" section
- Expand "Design Rationale" section
- Add "Composability" and "Consumer UX" sub-sections
- Describe versioning strategy
Update from Orbit Lens to include elaboration on various aspects and change name to stellar asset lists 
Rename Stellar Token Lists to Stellar Asset Lists to follow standard Stellar terminology
Add feedback, description, assets.domain fields
Add "Security Concerns" section
Expand "Design Rationale" section
Add "Composability" and "Consumer UX" sub-sections
Describe versioning strategy
Change some wording
Add SAL JSON schema, fix field description, cleanup formatting
Copy link
Member

@leighmcculloch leighmcculloch left a comment

Choose a reason for hiding this comment

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

Could you add the changelog section to set us up for keeping that updated, otherwise looks good to merge.

- [StellarExpert Top50](https://api.stellar.expert/explorer/public/asset-list/top50) asset list for Stellar Pubnet.
- [StellarExpert Testnet Top50](https://api.stellar.expert/explorer/testnet/asset-list/top50) asset list for Stellar
Testnet.

Copy link
Member

Choose a reason for hiding this comment

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

Can we add a changelog at the bottom, from the sep-template.md?

Suggested change
## Changelog
- `v0.2.0`: Initial draft. [#1440](https://github.com/stellar/stellar-protocol/pull/1440)

Copy link

This pull request is stale because it has been open for 30 days with no activity. It will be closed in 30 days unless the stale label is removed.

@github-actions github-actions bot added the stale label Mar 13, 2024
@silence48
Copy link
Contributor

This pull request is stale because it has been open for 30 days with no activity. It will be closed in 30 days unless the stale label is removed.

@orbitlens @leighmcculloch I had been away for a bit, but am back now... this PR should not go stale, maybe you can remove the label. What do we still need to do in order to implement this? What is the status of the mailing list discussion?

Thanks!

@orbitlens orbitlens changed the title Update SEP 0042 token lists SEP-0042 Stellar Asset Lists Standard Mar 27, 2024
@leighmcculloch leighmcculloch merged commit 27dc4cf into stellar:master Mar 27, 2024
3 checks passed
@orbitlens
Copy link
Contributor Author

Thank you, Leigh

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