Skip to content

Update wallet07.mdx #235

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

Merged
merged 28 commits into from
Oct 22, 2024
Merged

Update wallet07.mdx #235

merged 28 commits into from
Oct 22, 2024

Conversation

johnnygreeney
Copy link
Collaborator

Updated so this page now contains 2 queries as per request on https://cardanofoundation.atlassian.net/browse/LSY-160

Note indentation can appear skewed in github preview. Renders fine on localhost and live with nextra

@Sotatek-QuanLeA @darlisagc @satran004

Updated so this page now contains 2 queries as per request on https://cardanofoundation.atlassian.net/browse/LSY-160

### 7.1 Query all Stake Pools

To get started, the first step is to identify all stake pools. This SQL query retrieves information about active Cardano staking pools, retrieving the pool's ID, name and creation date.

Choose a reason for hiding this comment

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

"This SQL query retrieves information about active Cardano staking pools".
All stake pools also include retired, so can "active" be removed? This is the next query :)

@@ -65,7 +166,7 @@ Expected results format
<Steps>
### Select Specific Columns

In order to gather information about staking pools, the query focuses on:
Similar to our first part, we want to retrieve data for the following columns:

Choose a reason for hiding this comment

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

Maybe on the explanation we can just mentioned that query is similar to the previous one with the addition of the "Where" clause, which then we can explain?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@darlisagc updated now, please check, thanks

@johnnygreeney
Copy link
Collaborator Author

hi @darlisagc @satran004 this PR has been updated.

  • It includes fixes to relative paths (from September update)
  • Now also includes update to the 'README' (GitHub) and the 'Overview page' (https://cardano-foundation.github.io/cf-ledger-sync)
  • I added the animation as an MP4 file. Viewer must click 'full screen' to see it properly.

Please review, thanks

Copy link
Member

@satran004 satran004 left a comment

Choose a reason for hiding this comment

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

@satran004 satran004 merged commit 111b03b into cardano-foundation:main Oct 22, 2024
4 checks passed
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