Testnet + readme - Set to default to testnet, maximized readme #49
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
gregnazario
reviewed
Apr 15, 2025
|
|
||
| ## License | ||
|
|
||
| [MIT License](typescript/LICENSE) |
Contributor
There was a problem hiding this comment.
This is the Apache license, not the MIT license :)
Comment on lines
+1
to
+9
| # Apt-ID Interactive Learning Experience | ||
| ## Feasibility Analysis for May Hackathon Implementation | ||
|
|
||
| ## Executive Summary | ||
|
|
||
| This analysis assesses the feasibility of implementing the Apt-ID interactive learning experience for the May 5th workshop and subsequent May 12th hackathon. Based on our evaluation, we believe a **focused MVP version** of the experience is achievable within the timeline, though it will require immediate resource allocation and clearly defined scope limitations. | ||
|
|
||
| **Key Findings:** | ||
| - A simplified version focusing on contract modifications and UI generation is feasible |
Contributor
There was a problem hiding this comment.
Do you want to commit this?
Comment on lines
+1
to
+5
| # Apt-ID Interactive Learning Experience | ||
| ## Implementation Plan for May Hackathon | ||
|
|
||
| This document outlines the implementation strategy for embedding Apt-ID as an interactive learning experience for the May hackathon, focusing on a simplified approach where participants can make minor modifications to the existing contract and see immediate UI updates. | ||
|
|
Contributor
There was a problem hiding this comment.
Similarly the plans and all
Comment on lines
+1
to
+6
| # Apt ID | ||
|
|
||
|  | ||
|
|
||
| The codebase for [Apt ID](https://aptid.xyz), a profile based experience attached to an Aptos name. Built by Greg Nazario, with :heart: | ||
|
|
Contributor
There was a problem hiding this comment.
What's greadme vs README?
Author
There was a problem hiding this comment.
greadme is easter egg readme!
Comment on lines
+1
to
+6
| # Apt ID: Comprehensive Review | ||
|
|
||
| ## Overview | ||
|
|
||
| Apt ID is a decentralized identity platform built on the Aptos blockchain that functions similarly to Linktree. It allows users to create a profile linked to their Aptos Name Service (ANS) name, where they can add biographical information and links to various platforms. The project demonstrates several key features of the Aptos blockchain and Move programming language. | ||
|
|
Contributor
There was a problem hiding this comment.
This looks like a repeat
Author
There was a problem hiding this comment.
Yeah , let me clean up, i'll need a couple days
Comment on lines
1
to
+12
| import { Aptos, AptosConfig, Network as SdkNetwork } from "@aptos-labs/ts-sdk"; | ||
|
|
||
| export const NETWORK = "mainnet"; | ||
| export const CONTRACT_ADDRESS = "0x631f344549b798ad70cb5ab1842565b082fdfe488b7c6d56a257220222f6a191"; | ||
| // NETWORK CONFIGURATION | ||
| // TODO: For production/mainnet deployment, change these values back to: | ||
| // export const NETWORK = "mainnet"; | ||
| // network: SdkNetwork.MAINNET in the AptosConfig below | ||
| export const NETWORK = "testnet"; | ||
|
|
||
| // CONTRACT ADDRESS | ||
| // TODO: Update with actual testnet contract address after deployment | ||
| // For mainnet, use: 0x631f344549b798ad70cb5ab1842565b082fdfe488b7c6d56a257220222f6a191 | ||
| export const CONTRACT_ADDRESS = "0xaea5b834a6bcab2de9556051c200bd1dbc2f0c70f4eafced96f05ca0ff9af6dc"; |
Contributor
There was a problem hiding this comment.
These need to be using an environment variable to switch between the two.
Otherwise, it will break the autodeployed one, which uses mainnet.
gregnazario
reviewed
Jun 18, 2025
Contributor
gregnazario
left a comment
gregnazario
left a comment
There was a problem hiding this comment.
let's pull the analysis, plan, critique stuff out and then we can use it
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Buffed the readme to include quotes from greg, technical highlights, screenshot, detailed dev setup info (and a note about how to change it to mainnet from testnet)
other key changes include: