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

Update near accounts docs #154

Merged
merged 1 commit into from
Aug 23, 2022
Merged

Update near accounts docs #154

merged 1 commit into from
Aug 23, 2022

Conversation

mangas
Copy link
Contributor

@mangas mangas commented Jun 23, 2022

No description provided.

@mangas mangas requested a review from a team as a code owner June 23, 2022 11:32
@@ -62,6 +62,13 @@ dataSources:
source:
account: app.good-morning.near # This data source will monitor this account
startBlock: 10662188 # Required for NEAR
accounts:
Copy link
Contributor

Choose a reason for hiding this comment

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

I think let's pull this out of the example manifest

- NEAR data sources introduce an optional `source.accounts` field, which contains optional suffixes and prefixes.
At least prefix or suffix must be specified, they will match the any account starting or ending with the list of values respectively.
For the provided example it would match: `[app|good].*[morning.near|morning.testnet]`. If only a list of prefixes or suffixes is necessary
the other field can be omitted.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think let's drop the example above here in a code block

- good
suffixes:
- morning.near
- morning.testnet
Copy link
Contributor

Choose a reason for hiding this comment

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

I think remove the testnet from here, as it's mixing networks which isn't technically possible, so this is potentially confusing

@azf20
Copy link
Contributor

azf20 commented Jul 27, 2022

Could we rebase this for the new structure @mangas ?

@mangas mangas force-pushed the update-near-docs-accounts branch from 0bc0b2b to c0cc1c9 Compare August 17, 2022 14:11
@mangas mangas force-pushed the update-near-docs-accounts branch from c0cc1c9 to f518ae4 Compare August 23, 2022 09:19
Copy link
Contributor

@azf20 azf20 left a comment

Choose a reason for hiding this comment

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

Thanks!

@azf20 azf20 merged commit 8790a1b into main Aug 23, 2022
@azf20 azf20 deleted the update-near-docs-accounts branch August 23, 2022 09:27
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