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

Feat/restructure nav #536

Merged
merged 6 commits into from
Aug 2, 2024
Merged

Feat/restructure nav #536

merged 6 commits into from
Aug 2, 2024

Conversation

AnjaMis
Copy link
Contributor

@AnjaMis AnjaMis commented Jul 29, 2024

Adds new page layout for docs

next step: add getting started page

Summary by CodeRabbit

  • New Features

    • Introduced new documentation sections for connectors, contributions, and ecosystem creation, enhancing user navigation and resource accessibility.
    • Added placeholder sections for "Provide Computation" and "Share Resources" to facilitate future content updates.
  • Bug Fixes

    • Streamlined the FAQ and other guidance documents for improved clarity and user understanding of the Axone protocol.
  • Documentation

    • Updated various existing document structures, enhancing readability and user experience.
    • Renamed and reorganized sidebar categories to reflect current content strategy effectively.
  • Chores

    • Removed several navigation items from the configuration to streamline the user interface.

Copy link

vercel bot commented Jul 29, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 31, 2024 9:30am

Copy link
Contributor

coderabbitai bot commented Jul 29, 2024

Walkthrough

The recent updates enhance the organizational structure of the documentation by introducing new categories, refining existing content, and improving navigation. Key changes include the addition of several overview files for connectors and resource sharing, updates to the sidebar configuration, and a rebranding from "Tutorials" to "Guides." This restructuring aims to improve user experience, making it easier for community members to find relevant information and engage with the Axone project.

Changes

File Path Change Summary
docs/connector/_category_.json, docs/connector/overview.mdx New category and overview for "Connectors" added.
docs/contribute/_category_.json, docs/contribute/contribute.mdx New category for contributions and a guide for engaging with the Axone community introduced.
docs/create-ecosystem/_category_.json, docs/create-ecosystem/overview.mdx New category and overview for creating an ecosystem added.
docs/faq/_category_.json, docs/faq/faq.md FAQ category label changed to "FAQ" and position updated; content restructured for clarity.
docs/introduction/_category_.json, docs/introduction/installation.md, docs/introduction/protocol-concepts.md, docs/introduction/resources.mdx, docs/introduction/rules.mdx, docs/introduction/who-is-it-for.mdx, docs/introduction/workflows.mdx New introductory category created; several documents reorganized and refined for better clarity and presentation.
docs/nodes/_category_.json Category label changed from "Nodes & Validators" to "Running a Node."
docs/tutorials/_category_.json, docs/tutorials/cli-1.mdx, docs/tutorials/explorer-sc.md, docs/tutorials/keplr-1.mdx, docs/tutorials/prolog-1.mdx Tutorials renamed to "Guides"; various position adjustments and content refinements performed.
docusaurus.config.js, sidebars.js Navigation items removed and reorganized in the sidebar for improved clarity and accessibility.
docs/technical-documentation/_category_.json, docs/technical-documentation/technical-implementation/_category_.json, docs/technical-documentation/technical-implementation/decentralized-identity.mdx, docs/technical-documentation/technical-implementation/describe-resource.mdx, docs/technical-documentation/technical-implementation/describe-zone.mdx, docs/technical-documentation/technical-implementation/resource-governance.mdx, docs/technical-documentation/technical-implementation/verifiable-claims.mdx, docs/technical-documentation/technical-implementation/zone-governance.mdx Technical documentation reorganized under "Architecture"; various content updates for clarity and structure.

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant Documentation
    participant Sidebar

    User->>Sidebar: Requests Overview
    Sidebar->>Documentation: Fetches Updated Categories
    Documentation-->>Sidebar: Returns New Structure
    Sidebar-->>User: Displays Updated Navigation
Loading

🐰 In a world of docs so neat,
Changes hop, and users greet.
New paths to share, to build, to learn,
With every click, new ideas churn!
Explore we will, with joy and cheer,
Axone's wonders drawing near! 🌟


Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

Share
Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai generate interesting stats about this repository and render them as a table.
    • @coderabbitai show all the console.log statements in this repository.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (invoked as PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Additionally, you can add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@AnjaMis AnjaMis force-pushed the feat/restructure-nav branch from da91bda to 11d12ab Compare July 31, 2024 09:16
Copy link
Member

@bot-anik bot-anik left a comment

Choose a reason for hiding this comment

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

Device URL
desktop http://localhost:3000/

Device URL
mobile http://localhost:3000/

Not what you expected? Are your scores flaky? GitHub runners could be the cause.
Try running on Foo instead

@AnjaMis AnjaMis marked this pull request as ready for review July 31, 2024 09:31
@AnjaMis AnjaMis requested review from ccamel and amimart July 31, 2024 09:33
@AnjaMis AnjaMis self-assigned this Jul 31, 2024
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

Outside diff range, codebase verification and nitpick comments (1)
docs/introduction/resources.mdx (1)

16-16: Add a period after "etc".

In American English, abbreviations like "etc." require a period.

- etc…
+ etc.
Review details

Configuration used: .coderabbit.yaml
Review profile: CHILL

Commits

Files that changed from the base of the PR and between a35861e and 11d12ab.

Files selected for processing (38)
  • docs/connector/category.json (1 hunks)
  • docs/connector/overview.mdx (1 hunks)
  • docs/contribute/category.json (1 hunks)
  • docs/contribute/contribute.mdx (1 hunks)
  • docs/create-ecosystem/category.json (1 hunks)
  • docs/create-ecosystem/overview.mdx (1 hunks)
  • docs/faq/category.json (1 hunks)
  • docs/faq/faq.md (15 hunks)
  • docs/index.mdx (2 hunks)
  • docs/introduction/category.json (1 hunks)
  • docs/introduction/ai-factory.mdx (6 hunks)
  • docs/introduction/installation.md (1 hunks)
  • docs/introduction/protocol-concepts.md (2 hunks)
  • docs/introduction/resources.mdx (3 hunks)
  • docs/introduction/rules.mdx (1 hunks)
  • docs/introduction/who-is-it-for.mdx (7 hunks)
  • docs/introduction/workflows.mdx (4 hunks)
  • docs/nodes/category.json (1 hunks)
  • docs/provide-computation/category.json (1 hunks)
  • docs/provide-computation/overview.mdx (1 hunks)
  • docs/share-resources/category.json (1 hunks)
  • docs/share-resources/overview.mdx (1 hunks)
  • docs/technical-documentation/category.json (1 hunks)
  • docs/technical-documentation/technical-implementation/category.json (1 hunks)
  • docs/technical-documentation/technical-implementation/decentralized-identity.mdx (6 hunks)
  • docs/technical-documentation/technical-implementation/describe-resource.mdx (5 hunks)
  • docs/technical-documentation/technical-implementation/describe-zone.mdx (5 hunks)
  • docs/technical-documentation/technical-implementation/resource-governance.mdx (15 hunks)
  • docs/technical-documentation/technical-implementation/verifiable-claims.mdx (6 hunks)
  • docs/technical-documentation/technical-implementation/zone-governance.mdx (14 hunks)
  • docs/tutorials/category.json (1 hunks)
  • docs/tutorials/cli-1.mdx (1 hunks)
  • docs/tutorials/explorer-sc.md (1 hunks)
  • docs/tutorials/keplr-1.mdx (4 hunks)
  • docs/tutorials/prolog-1.mdx (2 hunks)
  • docs/whitepaper/category.json (1 hunks)
  • docusaurus.config.js (4 hunks)
  • sidebars.js (5 hunks)
Files skipped from review due to trivial changes (24)
  • docs/connector/category.json
  • docs/connector/overview.mdx
  • docs/contribute/category.json
  • docs/contribute/contribute.mdx
  • docs/create-ecosystem/category.json
  • docs/create-ecosystem/overview.mdx
  • docs/faq/category.json
  • docs/introduction/category.json
  • docs/introduction/installation.md
  • docs/introduction/protocol-concepts.md
  • docs/introduction/who-is-it-for.mdx
  • docs/nodes/category.json
  • docs/provide-computation/category.json
  • docs/provide-computation/overview.mdx
  • docs/share-resources/category.json
  • docs/share-resources/overview.mdx
  • docs/technical-documentation/category.json
  • docs/technical-documentation/technical-implementation/category.json
  • docs/technical-documentation/technical-implementation/verifiable-claims.mdx
  • docs/tutorials/category.json
  • docs/tutorials/cli-1.mdx
  • docs/tutorials/explorer-sc.md
  • docs/tutorials/prolog-1.mdx
  • docs/whitepaper/category.json
Additional context used
LanguageTool
docs/tutorials/keplr-1.mdx

[style] ~24-~24: Try using a synonym here to strengthen your writing.
Context: ...et access to your wallet. This password gives you access to your wallet, but if you lose ...

(GIVE_PROVIDE)


[grammar] ~121-~121: The word “checkout” is a noun. The verb is spelled with a space.
Context: ...n in Keplr, for example. - You can also checkout the explorer to see the transactions st...

(NOUN_VERB_CONFUSION)


[uncategorized] ~121-~121: It seems likely that a singular genitive (’s) apostrophe is missing.
Context: ...n also checkout the explorer to see the transactions status and more. Congratulations! You ...

(AI_HYDRA_LEO_APOSTROPHE_S_XS)

docs/introduction/rules.mdx

[style] ~16-~16: In American English, abbreviations like “etc.” require a period.
Context: ...("prohibited", "permitted", "possible", etc). Depending on the answer, the executio...

(ETC_PERIOD)

docs/technical-documentation/technical-implementation/decentralized-identity.mdx

[uncategorized] ~59-~59: This verb may not be in the correct form. Consider using a different form for this context.
Context: ... on transaction fees. ::: ### Example: apply for a job Issuer - University: _R...

(AI_EN_LECTOR_REPLACEMENT_VERB_FORM)


[grammar] ~90-~90: Did you mean “I”?
Context: ...organization, or entity they refer to. A DID applies to anything that has an ide...

(A_MD_VB)


[grammar] ~91-~91: After the auxiliary verb ‘do’, use the base form of a verb. Did you mean “apply”?
Context: ...zation, or entity they refer to. A DID applies to anything that has an identity. An en...

(AUXILIARY_DO_WITH_INCORRECT_VERB_FORM)


[uncategorized] ~211-~211: “its” (belonging to it) seems less likely than “it’s” (it is)
Context: ...in Axone Protocol The Axone ontology is ...

(AI_HYDRA_LEO_CPT_ITS_ITIS)


[style] ~230-~230: You can shorten this phrase to avoid wordiness.
Context: ...metadata of its resource - Holder as he is the owner of the resource - Registrant as he declare...

(BE_THE_MAKER_OF_WORDINESS)


[grammar] ~237-~237: The noun “third party” is written as two words.
Context: ...t can be the Provider, the Issuer, or a third-party

(THE_THIRD_PARTY)

docs/technical-documentation/technical-implementation/describe-zone.mdx

[grammar] ~47-~47: Use “the” before the superlative.
Context: ...s where the security of private keys is a highest priority. For secure keyring s...

(THE_SUPERLATIVE)


[typographical] ~115-~115: It appears that a comma is missing.
Context: ...ab-ai-zone-description.jsonld). :::info Would you like to describe your zone with oth...

(COMMA_BEFORE_QUESTION_WITH_MD)


[uncategorized] ~158-~158: Loose punctuation mark.
Context: ...](https://www.uuidgenerator.net). - id: copy the did:key of the zone - `hasDesc...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~163-~163: Loose punctuation mark.
Context: ...ed topic in the jsonld. - issuanceDate: update the date for Today - id (issue...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~187-~187: Loose punctuation mark.
Context: ...h: - /MY-DIRECTORY/MY-DATASET.jsonld : credential file address - MY_ADDR : i...

(UNLIKELY_OPENING_PUNCTUATION)


[grammar] ~199-~199: Did you mean “there are new fields”?
Context: ...ne-description.nq ``` You can see that there is new fields in the jsonld with the cryptographic pr...

(THERE_S_MANY)


[style] ~221-~221: For conciseness, try rephrasing this sentence.
Context: ...w in the hands of the Holder. Note that it is possible that the Issuer is also the Holder. The Axone blockchain ...

(MAY_MIGHT_BE)

docs/introduction/ai-factory.mdx

[style] ~94-~94: Three successive sentences begin with the same word. Consider rewording the sentence or use a thesaurus to find a synonym.
Context: ...hese rules are allowed by the Zone. All Resources and Services compatible with ...

(ENGLISH_WORD_REPEAT_BEGINNING_RULE)

docs/technical-documentation/technical-implementation/describe-resource.mdx

[uncategorized] ~9-~9: Use a comma before ‘and’ if it connects two independent clauses (unless they are closely connected and short).
Context: ...ice! You're a digital resource provider and you want to share a dataset or a servic...

(COMMA_COMPOUND_SENTENCE)


[grammar] ~50-~50: Use “the” before the superlative.
Context: ...s where the security of private keys is a highest priority. For secure keyring s...

(THE_SUPERLATIVE)


[typographical] ~122-~122: It appears that a comma is missing.
Context: ...al-service-description.jsonld). :::info Would you like to describe your dataset with ...

(COMMA_BEFORE_QUESTION_WITH_MD)


[uncategorized] ~123-~123: Possible missing comma found.
Context: ...'t directly add these properties in the DatasetDescriptionCredential as defined in the Ontology. ::: ### De...

(AI_HYDRA_LEO_MISSING_COMMA)


[style] ~132-~132: Consider a shorter alternative to avoid wordiness.
Context: ...y provided to the nearest hundred block in order to maintain privacy. This data is as accur...

(IN_ORDER_TO_PREMIUM)


[uncategorized] ~171-~171: Loose punctuation mark.
Context: ...](https://www.uuidgenerator.net). - id: copy the did:key of the dataset - `hasD...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~174-~174: Loose punctuation mark.
Context: ...format in the jsonld. - hasGeoCoverage: select the geographical zone concerned ...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~175-~175: Loose punctuation mark.
Context: ... selected zone in the jsonld. - hasTag: fill in a list of tags - `hasTemporalCo...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~179-~179: Loose punctuation mark.
Context: ...ed topic in the jsonld. - issuanceDate: update the date for Today - id (issue...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~226-~226: Loose punctuation mark.
Context: ...](https://www.uuidgenerator.net). - id: copy the did:key of the digital service...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~228-~228: Loose punctuation mark.
Context: ...format in the jsonld. - hasDescription: fill in the description of the service ...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~254-~254: Loose punctuation mark.
Context: ...h: - /MY-DIRECTORY/MY-DATASET.jsonld : credential file address - MY_ADDR : i...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~266-~266: The verb “is” doesn’t seem to fit in this context, “are” is probably more formally correct.
Context: ...cription.nq ``` You can see that there is new fields in the jsonld with the crypt...

(AI_HYDRA_LEO_CPT_IS_ARE)


[style] ~290-~290: For conciseness, try rephrasing this sentence.
Context: ...w in the hands of the Holder. Note that it is possible that the Issuer is also the Holder. The Axone blockchain ...

(MAY_MIGHT_BE)

docs/technical-documentation/technical-implementation/zone-governance.mdx

[uncategorized] ~32-~32: Possible missing preposition found.
Context: ...ere are the five steps to create a zone governance in the Dataverse: <div style={{ displa...

(AI_HYDRA_LEO_MISSING_OF)


[style] ~166-~166: ‘taken into account’ might be wordy. Consider a shorter alternative.
Context: ...on the particular conditions or context taken into account. It is relevant to conditional clauses,...

(EN_WORDINESS_PREMIUM_TAKEN_INTO_ACCOUNT)


[uncategorized] ~167-~167: Loose punctuation mark.
Context: ...s explicitly permitted. - prohibited: This modality expresses an explicit pro...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~267-~267: Possible missing comma found.
Context: ...43B Then you need to find the contract address which is necessary for the next step. Y...

(AI_HYDRA_LEO_MISSING_COMMA)


[uncategorized] ~410-~410: Loose punctuation mark.
Context: ...h: - /MY-DIRECTORY/MY-DATASET.jsonld : credential file address - MY_ADDR : i...

(UNLIKELY_OPENING_PUNCTUATION)


[grammar] ~422-~422: Did you mean “there are new fields”?
Context: ...one-governance.nq ``` You can see that there is new fields in the jsonld with the cryptographic pr...

(THERE_S_MANY)


[style] ~449-~449: For conciseness, try rephrasing this sentence.
Context: ...w in the hands of the Holder. Note that it is possible that the Issuer is also the Holder. The Axone blockchain ...

(MAY_MIGHT_BE)

docs/faq/faq.md

[style] ~10-~10: For conciseness, consider replacing this expression with an adverb.
Context: ...ck of trust and misaligned incentives. Right now, there's no infrastructure, centralized...

(AT_THE_MOMENT)


[style] ~17-~17: To make your text as clear as possible to all readers, do not use this foreign term. Possible alternatives are “below” or “further on” (in a document).
Context: ...tems. 3. Agnostic: any data, model, infra or API can be shared and orchestrated. ...

(INFRA)


[uncategorized] ~100-~100: Possible missing comma found.
Context: ... service. It goes way beyond what Ocean does as it has new primitives to build anyth...

(AI_HYDRA_LEO_MISSING_COMMA)


[uncategorized] ~175-~175: Possible missing comma found.
Context: ...ypes. ## What is the Cognitarium smart contract in a few words? The Cognitarium smart ...

(AI_HYDRA_LEO_MISSING_COMMA)


[uncategorized] ~189-~189: Possible missing comma found.
Context: ...is broken. ## What is the Pactum smart contract in a few words? The Pactum smart contr...

(AI_HYDRA_LEO_MISSING_COMMA)


[uncategorized] ~195-~195: Possible missing comma found.
Context: ...ves many parties. ## What is the Logic Module in a few words? The Logic module is de...

(AI_HYDRA_LEO_MISSING_COMMA)

docs/technical-documentation/technical-implementation/resource-governance.mdx

[style] ~186-~186: ‘taken into account’ might be wordy. Consider a shorter alternative.
Context: ...on the particular conditions or context taken into account. It is relevant to conditional clauses,...

(EN_WORDINESS_PREMIUM_TAKEN_INTO_ACCOUNT)


[uncategorized] ~187-~187: Loose punctuation mark.
Context: ...s explicitly permitted. - prohibited: This modality expresses an explicit pro...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~286-~286: Possible missing comma found.
Context: ...1A6 Then you need to find the contract address which is necessary for the next step. Y...

(AI_HYDRA_LEO_MISSING_COMMA)


[uncategorized] ~429-~429: Loose punctuation mark.
Context: ...h: - /MY-DIRECTORY/MY-DATASET.jsonld : credential file address - MY_ADDR : i...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~441-~441: The verb “is” doesn’t seem to fit in this context, “are” is probably more formally correct.
Context: ...vernance.nq ``` You can see that there is new fields in the jsonld with the crypt...

(AI_HYDRA_LEO_CPT_IS_ARE)


[style] ~468-~468: For conciseness, try rephrasing this sentence.
Context: ...w in the hands of the Holder. Note that it is possible that the Issuer is also the Holder. The Axone blockchain ...

(MAY_MIGHT_BE)

Gitleaks
docs/technical-documentation/technical-implementation/decentralized-identity.mdx

137-137: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)

docs/technical-documentation/technical-implementation/describe-zone.mdx

97-97: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


103-103: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


141-141: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


149-149: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)

docs/technical-documentation/technical-implementation/describe-resource.mdx

97-97: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


103-103: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


109-109: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


151-151: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


162-162: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


207-207: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


217-217: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)

docs/technical-documentation/technical-implementation/zone-governance.mdx

76-76: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


323-323: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


341-341: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


386-386: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)

docs/technical-documentation/technical-implementation/resource-governance.mdx

342-342: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


405-405: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)

Additional comments not posted (73)
docs/index.mdx (4)

8-8: LGTM! Title change approved.

The title update to "About Axone" aligns with the new focus on Axone's functionalities and applications in the AI domain.


10-17: LGTM! Introduction update approved.

The new introduction provides a clearer and more detailed description of Axone's purpose and capabilities, highlighting key technologies and value propositions.


31-42: LGTM! New sections and cards approved.

The new sections and cards improve the user experience by guiding users to relevant resources for builders and data scientists.


39-42: LGTM! "Data Scientists" card content approved.

The card provides a clear call to action for data scientists to share their resources, aligning with the overall goal of enhancing user engagement.

sidebars.js (9)

6-17: LGTM! Category renaming approved.

The category "Whitepaper" has been renamed to "Introduction," aligning with the shift in focus from a formal whitepaper to a more introductory context.


18-29: LGTM! Category renaming approved.

The category "Nodes & Validators" has been renamed to "Share Resources," suggesting a broader scope that may encompass more than just nodes and validators.


30-41: LGTM! Category renaming approved.

The category "Academy" has been replaced with "Create an Ecosystem," aligning with the new focus on creating ecosystems within the Axone platform.


42-53: LGTM! New category approved.

The new category "Provide Computation" adds content related to computational resources, enhancing the documentation's scope.


54-65: LGTM! New category approved.

The new category "Connector" adds content related to connectors, improving the documentation's organization.


Line range hint 69-104: LGTM! Category renaming approved.

The category "Technical documentation" has been renamed to "Architecture," aligning with the new focus on architectural aspects within the Axone platform.


115-118: LGTM! New links approved.

The new links under the "Architecture" category enhance the accessibility and organization of the documentation.


137-148: LGTM! Category reintroduction approved.

The "nodes" category has been reintroduced, aligning with the need for comprehensive documentation on node operation.


174-185: LGTM! New category approved.

The new category "Contribute to Axone" encourages community contributions, enhancing the collaborative aspect of the project.

docs/tutorials/keplr-1.mdx (5)

2-2: LGTM! Sidebar position change approved.

The sidebar position was adjusted from 2 to 1, likely enhancing the tutorial's visibility.


19-24: LGTM! Instruction clarification approved.

The clarification improves the tutorial's readability and ensures users understand the importance of securely storing the recovery phrase.

Tools
LanguageTool

[style] ~24-~24: Try using a synonym here to strengthen your writing.
Context: ...et access to your wallet. This password gives you access to your wallet, but if you lose ...

(GIVE_PROVIDE)


31-31: LGTM! Instruction addition approved.

The addition provides a useful tip for users, enhancing their experience with the Keplr Wallet.


55-62: LGTM! Instruction clarification approved.

The clarification improves the tutorial's readability and ensures users can easily check their $AXONE balance.


69-73: LGTM! Instruction clarification approved.

The clarification ensures users can easily verify their $AXONE balance within the Keplr Wallet.

docs/introduction/resources.mdx (3)

Line range hint 20-42:
LGTM!

The section on Digital Services is well-structured and clear.


44-60: LGTM!

The section on Verifiable Claims is clear and concise.


Line range hint 62-112:
LGTM!

The section on Ontology is well-structured and clear.

docs/introduction/rules.mdx (4)

5-17: LGTM!

The section on defining rules is clear and concise.

Tools
LanguageTool

[style] ~16-~16: In American English, abbreviations like “etc.” require a period.
Context: ...("prohibited", "permitted", "possible", etc). Depending on the answer, the executio...

(ETC_PERIOD)


18-29: LGTM!

The section on types of rules is well-structured and clear.


30-33: LGTM!

The section on Prolog in Axone is clear and concise.


34-112: LGTM!

The section on resource consents is well-structured and clear.

docusaurus.config.js (2)

89-90: LGTM!

The removal of navigation items simplifies the navigation structure, aligning with the PR's objective.


89-90: LGTM!

The formatting change in the editUrl function improves readability without altering functionality.

docs/introduction/workflows.mdx (3)

Line range hint 5-30:
LGTM!

The introductory content on workflows within the Axone ecosystem is clear and well-structured.


Line range hint 31-278:
LGTM!

The "Workflow lifecycle" section and subsequent content are well-structured and enhance understanding of workflows within the Axone protocol.

Tools
LanguageTool

[style] ~62-~62: ‘at this juncture’ might be wordy. Consider a shorter alternative.
Context: ...tor who directs each participant. It is at this juncture that the concept of Orchestration comes...

(EN_WORDINESS_PREMIUM_AT_THIS_JUNCTURE)


[uncategorized] ~66-~66: Possible missing comma found.
Context: ...fic resource takes on the orchestration role while the protocol maintains its positi...

(AI_HYDRA_LEO_MISSING_COMMA)


2-2: Verify sidebar position update.

The sidebar position has been updated from 3 to 5. Ensure this change aligns with the overall navigation structure.

Verification successful

Sidebar position update verified.

The sidebar_position for docs/introduction/workflows.mdx is correctly set to 5, as intended.

  • docs/introduction/workflows.mdx: sidebar_position: 5
Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Verify the sidebar position update in the navigation structure.

# Test: Search for sidebar position configurations. Expect: Sidebar position for workflows set to 5.
rg --type mdx -A 5 $'sidebar_position'

Length of output: 65


Script:

#!/bin/bash
# Description: Verify the sidebar position update in the navigation structure.

# Test: Search for sidebar position configurations in .mdx files. Expect: Sidebar position for workflows set to 5.
rg --glob '*.mdx' -A 5 'sidebar_position'

Length of output: 9835

docs/technical-documentation/technical-implementation/decentralized-identity.mdx (5)

10-14: LGTM!

The streamlined introductory content enhances readability and clarity.


29-48: LGTM!

The restructured section on decentralized identity roles improves readability and understanding.


Line range hint 59-87:
LGTM!

The refinements in the example section enhance clarity.

Tools
LanguageTool

[uncategorized] ~59-~59: This verb may not be in the correct form. Consider using a different form for this context.
Context: ... on transaction fees. ::: ### Example: apply for a job Issuer - University: _R...

(AI_EN_LECTOR_REPLACEMENT_VERB_FORM)


Line range hint 89-239:
LGTM!

The refinements in the section on DIDs and subsequent content improve clarity and understanding.

Tools
LanguageTool

[grammar] ~202-~202: The auxiliary verb ‘do’ requires the base form of the verb.
Context: ..., ECDSA, Ed25519, etc.). A did:key DID looks like this: did:key:z12ab34cd56ef78gh90i...

(DID_BASEFORM)


[uncategorized] ~211-~211: “its” (belonging to it) seems less likely than “it’s” (it is)
Context: ...in Axone Protocol The Axone ontology is ...

(AI_HYDRA_LEO_CPT_ITS_ITIS)


[style] ~230-~230: You can shorten this phrase to avoid wordiness.
Context: ...metadata of its resource - Holder as he is the owner of the resource - Registrant as he declare...

(BE_THE_MAKER_OF_WORDINESS)


[grammar] ~237-~237: The noun “third party” is written as two words.
Context: ...t can be the Provider, the Issuer, or a third-party

(THE_THIRD_PARTY)

Gitleaks

202-203: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


5-6: Verify import usage.

The import statements for Quiz component and associated JSON resource have been added. Ensure these imports are used correctly in the document.

Verification successful

Let's correct the script to search for the usage of the Quiz component in .mdx files without specifying a file type.


Imports verified successfully.

The Quiz component and the associated JSON resource are correctly imported and used within the decentralized-identity.mdx file.

  • docs/technical-documentation/technical-implementation/decentralized-identity.mdx
Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Verify the usage of imported `Quiz` component and associated JSON resource.

# Test: Search for the usage of `Quiz` component. Expect: Correct usage in the document.
rg --type mdx -A 5 $'Quiz'

Length of output: 53


Script:

#!/bin/bash
# Description: Verify the usage of imported `Quiz` component and associated JSON resource.

# Test: Search for the usage of `Quiz` component. Expect: Correct usage in the document.
rg 'Quiz' -A 5

Length of output: 3371

docs/technical-documentation/technical-implementation/describe-zone.mdx (6)

9-12: LGTM!

The refined introductory content enhances clarity and readability.


24-33: LGTM!

The refinements in the prerequisite section enhance clarity and ensure that the prerequisites are easy to follow.


Line range hint 38-75:
LGTM!

The refinements in the "Create keys" section improve understanding and emphasize security considerations.

Tools
LanguageTool

[style] ~7-~7: ‘taking into account’ might be wordy. Consider a shorter alternative.
Context: ...ources are compatible with these rules, taking into account the associated consents. A resource is ...

(EN_WORDINESS_PREMIUM_TAKING_INTO_ACCOUNT)


[grammar] ~47-~47: Use “the” before the superlative.
Context: ...s where the security of private keys is a highest priority. For secure keyring s...

(THE_SUPERLATIVE)


Line range hint 111-278:
LGTM!

The refinements in the section on creating credentials and subsequent content improve clarity and understanding.

Tools
LanguageTool

[typographical] ~115-~115: It appears that a comma is missing.
Context: ...ab-ai-zone-description.jsonld). :::info Would you like to describe your zone with oth...

(COMMA_BEFORE_QUESTION_WITH_MD)


[uncategorized] ~158-~158: Loose punctuation mark.
Context: ...](https://www.uuidgenerator.net). - id: copy the did:key of the zone - `hasDesc...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~163-~163: Loose punctuation mark.
Context: ...ed topic in the jsonld. - issuanceDate: update the date for Today - id (issue...

(UNLIKELY_OPENING_PUNCTUATION)


[uncategorized] ~187-~187: Loose punctuation mark.
Context: ...h: - /MY-DIRECTORY/MY-DATASET.jsonld : credential file address - MY_ADDR : i...

(UNLIKELY_OPENING_PUNCTUATION)


[grammar] ~199-~199: Did you mean “there are new fields”?
Context: ...ne-description.nq ``` You can see that there is new fields in the jsonld with the cryptographic pr...

(THERE_S_MANY)

Gitleaks

97-97: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


103-103: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


141-141: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


149-149: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


76-77: Verify key creation process.

Ensure that the key creation process aligns with the security guidelines provided.

Verification successful

Key creation process verified.

The documentation correctly instructs users to store mnemonic phrases safely, aligning with security guidelines.

  • commands_versioned_docs/version-v7.1.0/okp4d_keys_add.md
  • docs/tutorials/prolog-1.mdx
  • docs/technical-documentation/technical-implementation/describe-zone.mdx
Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Verify the key creation process.

# Test: Search for key creation commands. Expect: Correct usage of key creation commands.
rg --type mdx -A 5 $'okp4d keys add'

Length of output: 65


Script:

#!/bin/bash
# Description: Verify the key creation process.

# Test: Search for key creation commands. Expect: Correct usage of key creation commands.
rg -A 5 'okp4d keys add'

Length of output: 9280


Line range hint 2-2:
Verify sidebar position update.

The sidebar position has been updated to 5. Ensure this change aligns with the overall navigation structure.

Verification successful

Verified sidebar position update.

The sidebar position for docs/technical-documentation/technical-implementation/describe-zone.mdx has been correctly set to 5, aligning with the overall navigation structure.

  • docs/technical-documentation/technical-implementation/describe-zone.mdx: sidebar_position: 5
Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Verify the sidebar position update in the navigation structure.

# Test: Search for sidebar position configurations. Expect: Sidebar position for zones set to 5.
rg --type mdx -A 5 $'sidebar_position'

Length of output: 65


Script:

#!/bin/bash
# Description: Verify the sidebar position update in the navigation structure.

# Test: Search for sidebar position configurations. Expect: Sidebar position for zones set to 5.
rg --glob '*.mdx' -A 5 'sidebar_position'

Length of output: 9835

Tools
LanguageTool

[style] ~7-~7: ‘taking into account’ might be wordy. Consider a shorter alternative.
Context: ...ources are compatible with these rules, taking into account the associated consents. A resource is ...

(EN_WORDINESS_PREMIUM_TAKING_INTO_ACCOUNT)


[grammar] ~47-~47: Use “the” before the superlative.
Context: ...s where the security of private keys is a highest priority. For secure keyring s...

(THE_SUPERLATIVE)

docs/introduction/ai-factory.mdx (5)

2-2: Verify the impact of sidebar position change.

The sidebar position has been updated from 6 to 9. Ensure that this change aligns with the new navigation structure and does not disrupt the user experience.


5-6: LGTM! Simplified import statements.

The import statements have been simplified by removing unnecessary semicolons, enhancing code clarity.


28-30: LGTM! Improved readability of AI Factory components list.

The list of AI Factory components has been updated with a more structured format, enhancing readability and clarity.


62-65: LGTM! Enhanced structure and clarity in identity management section.

The identity management section has been reformatted for better structure and clarity, improving readability.


217-217: LGTM! Consistent JSX formatting.

The <Quiz quiz={quiz} /> component has been updated for consistency, following best practices for JSX formatting in React.

docs/technical-documentation/technical-implementation/describe-resource.mdx (5)

7-7: LGTM! Improved flow of introductory paragraph.

The introductory paragraph has been slightly altered to improve the flow of information, enhancing readability and engagement.


13-18: LGTM! Reformatted HTML elements for better readability.

The HTML elements for displaying an image have been reformatted for better readability, improving visual clarity and structure.


21-32: LGTM! Clearer formatting of prerequisites list.

The list of prerequisites has been reformatted for clearer formatting, enhancing the document's usability.


144-164: LGTM! Consistent JSON-LD structure for dataset description.

The JSON-LD structure for dataset description has been updated for consistency, ensuring a clear and usable metadata representation.

Tools
Gitleaks

151-151: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


162-162: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


200-219: LGTM! Consistent JSON-LD structure for digital service description.

The JSON-LD structure for digital service description has been updated for consistency, ensuring a clear and usable metadata representation.

Tools
Gitleaks

207-207: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


217-217: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)

docs/technical-documentation/technical-implementation/zone-governance.mdx (5)

8-8: LGTM! Simplified document by removing reading time indication.

The reading time indication was removed, simplifying the document and focusing on content.


24-29: LGTM! Reformatted HTML elements for better readability.

The HTML elements for displaying an image have been reformatted for better readability, improving visual clarity and structure.


34-39: LGTM! Reformatted HTML elements for better readability.

The HTML elements for displaying an image have been reformatted for better readability, improving visual clarity and structure.


57-67: LGTM! Streamlined governance structure section for clarity.

The section detailing the governance structure has been streamlined for clarity, enhancing readability and usability.


316-388: LGTM! Consistent JSON-LD structure for governance text credential.

The JSON-LD structure for governance text credential has been updated for consistency, ensuring a clear and usable metadata representation.

Tools
Gitleaks

323-323: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


341-341: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)


386-386: Detected a Generic API Key, potentially exposing access to various services and sensitive operations.

(generic-api-key)

docs/faq/faq.md (10)

7-7: Title change enhances clarity.

The new title "What problem is Axone solving?" is more specific and directly addresses the core challenges Axone aims to solve.


9-11: Streamlined explanation improves clarity.

The streamlined explanation effectively communicates the core issues related to collaborative AI training, the lack of infrastructure for resource sharing, and the need for governance.

Tools
LanguageTool

[style] ~10-~10: For conciseness, consider replacing this expression with an adverb.
Context: ...ck of trust and misaligned incentives. Right now, there's no infrastructure, centralized...

(AT_THE_MOMENT)


13-19: Condensed description enhances focus.

The condensed description highlights Axone's role in facilitating interoperability among digital resources, focusing on its unique features and benefits.

Tools
LanguageTool

[style] ~17-~17: To make your text as clear as possible to all readers, do not use this foreign term. Possible alternatives are “below” or “further on” (in a document).
Context: ...tems. 3. Agnostic: any data, model, infra or API can be shared and orchestrated. ...

(INFRA)


21-23: New section provides clear explanation.

The section "What is trust-minimized data sharing?" clearly explains the concept and its importance in the context of Axone.


25-33: New section provides detailed explanation.

The section "What is a Zone?" provides a detailed explanation of the concept of Zones and their role in the Axone ecosystem.


Line range hint 33-46: New section categorizes resources clearly.

The section "What types of resources can be shared?" categorizes resources and services, providing clarity on the types of resources that can be shared within Axone.

Tools
LanguageTool

[style] ~10-~10: For conciseness, consider replacing this expression with an adverb.
Context: ...ck of trust and misaligned incentives. Right now, there's no infrastructure, centralized...

(AT_THE_MOMENT)


[style] ~17-~17: To make your text as clear as possible to all readers, do not use this foreign term. Possible alternatives are “below” or “further on” (in a document).
Context: ...tems. 3. Agnostic: any data, model, infra or API can be shared and orchestrated. ...

(INFRA)


Line range hint 46-55: New section explains important distinction.

The section "Why do you distinguish Zones and digital resources?" explains the distinction between Zones and resources, highlighting the flexibility and adaptability of the Axone ecosystem.


55-59: New section clarifies storage mechanism.

The section "How does Axone store the resources and services?" clarifies that Axone does not store resources but references them, explaining the mechanism of making resources available to the network.


Line range hint 59-69: New section details referencing process.

The section "How does the referencing process work in Axone?" provides a detailed explanation of the referencing process, including consent rules, on-chain enforcement, and metadata management.

Tools
LanguageTool

[style] ~53-~53: The phrase “a variety of” may be wordy. To make your writing clearer, consider replacing it.
Context: ... scalable, and capable of accommodating a variety of rules and conditions for each Zone, the...

(A_VARIETY_OF)


69-79: New section explains ontology concept.

The section "What is an ontology?" explains the concept of ontology in the context of Axone, detailing its role in describing metadata and facilitating understanding and interaction with resources.

docs/technical-documentation/technical-implementation/resource-governance.mdx (7)

8-8: Removal of <i> tag improves readability.

The removal of the <i> tag for reading time simplifies the document and improves readability.


24-29: Simplified HTML attributes enhance readability.

The simplification of HTML attributes enhances the document's readability and maintainability.


34-39: Simplified HTML attributes enhance readability.

The simplification of HTML attributes enhances the document's readability and maintainability.


48-57: New section provides clear prerequisites.

The section "Prerequisite" provides clear instructions on the prerequisites for the tutorial, improving the user's understanding and preparation.


60-86: New section details governance definition.

The section "Step 1: Define the governance of your resource" provides a detailed explanation of how to define the governance rules of a resource, including a hierarchical text structure.


90-98: New section provides clear example.

The governance rules for the service IPFS provide a clear example, enhancing the user's understanding.


Line range hint 104-225: New section details Prolog program creation.

The section "Step 2: Create the Prolog program" provides a detailed explanation of how to create a Prolog program for governance, including hierarchical elements and legal terms.

docs/tutorials/keplr-1.mdx Show resolved Hide resolved
Copy link
Member

@ccamel ccamel left a comment

Choose a reason for hiding this comment

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

👍

@AnjaMis AnjaMis merged commit 053d478 into main Aug 2, 2024
14 checks passed
@AnjaMis AnjaMis deleted the feat/restructure-nav branch August 2, 2024 13:32
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