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

Dead / unavailable links #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

diegoxter
Copy link

No description provided.

@@ -234,7 +234,7 @@ The string, math, and table packages are available. However, you can not use the

If the smart contract is handling simple types of data, it would not be difficult to implement using only the basic APIs (getItem(), setItem()). However, complex data structures, data association, scope queries, filtering, sorting, and other features require the complexity and size of the data logic so developers can not focus on critical business logic. To solve this problem, Aergo supports SQL. This section details the types and usage of SQL APIs available in smart contracts

> Note: The db package is currently only available on private networks and publicly on [SQL TestNet](https://sqltestnet.aergoscan.io/).
> Note: The db package is currently only available on private networks and publicly on [SQL TestNet](https://sqltestnet.aergoscan.io/) (currently unavailable).
Copy link
Member

@kroggen kroggen Dec 6, 2024

Choose a reason for hiding this comment

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

Note: The db package is currently only available on private networks and publicly on AlphaNet (test network for private chains)

@@ -516,7 +516,7 @@ abi.register_view(sendAergo)

Aergo smart contract has `db` library that supports SQL features.

> Note: The db package is currently only available on private networks and publicly on [SQL TestNet](https://sqltestnet.aergoscan.io/).
> Note: The db package is currently only available on private networks and publicly on [SQL TestNet](https://sqltestnet.aergoscan.io/) (currently unavailable).
Copy link
Member

Choose a reason for hiding this comment

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

Note: The db package is currently only available on private networks and publicly on AlphaNet (test network for private chains)

@@ -5,7 +5,7 @@ The genesis block is the first block created on a new chain according to a prede
If you want to bootstrap your own Aergo blockchain, you will need to configure and create the genesis block.
All nodes producing blocks and syncing with the network need to use the same genesis block.

You can find an example genesis configuration json file `here <https://github.com/aergoio/aergo-docker/blob/1ad16cf7881d9ba8f2efc350cf609c9416e76666/node/testnet-genesis.json>`_.
You can find an example genesis configuration json file `here <https://github.com/aergoio/aergo-docker/blob/1ad16cf7881d9ba8f2efc350cf609c9416e76666/node/testnet-genesis.json>`_ (dead).
Copy link
Member

Choose a reason for hiding this comment

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

Oh, the repo is private. This is why it works for us internally.
We will solve it. Thanks for sharing

@@ -70,7 +70,7 @@ List of Subprotocols
--------------------

Code is hexadecimal number.
Refer to `Subprotocols <subprotocols.html>`_ for detailed information of each subprotocol.
Refer to `Subprotocols <subprotocols.html>`_ (dead) for detailed information of each subprotocol.
Copy link
Member

Choose a reason for hiding this comment

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

I suspect that this line can be removed

@kroggen
Copy link
Member

kroggen commented Dec 6, 2024

The BlockProducedNotice appears twice. One of them can be removed.
Also the display is incorrect with the last one, probably because it lacks the lines

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