docs: improve developer onboarding in README#409
Closed
fttawa wants to merge 1 commit into
Closed
Conversation
PR Gatekeeper
This PR has been automatically closed. Follow the process in CONTRIBUTING.md and try again. |
Welcome @fttawa!Thanks for your first PR to La Tanda. Before review, please confirm:
Resources
A maintainer will review your PR within 24-48 hours. Automated message for first-time contributors |
4 tasks
Author
|
Acknowledged the gatekeeper result: #50 is not assigned to me yet, so I am leaving this PR closed for now instead of repeatedly reopening it. I posted a standard |
Author
|
Checklist response to the bot prompt:
The current blocker remains the gatekeeper requirement that issue #50 must be assigned to me before review. I will reopen/resubmit after assignment. |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Development Setupsection toREADME.mdwith the local static-server flow usingnpx serve ..Project Structureoverview that matches the actual repository layout.README.mdonly and links to the existing Swagger UI, Dev Portal, and Chain Explorer instead of duplicating the Dev Portal content.Codebase verification answer
marketplace-social.htmlloads the root-levelmarketplace-social.jsvia<script src="marketplace-social.js?...">.js/marketplace-social.js, but the marketplace page uses the file at the HTML root.api-proxy.js;api-proxy-enhanced.jscontains the larger enhanced simulation layer.Validation
git diff --checkrgforDevelopment Setup,npx serve .,Project Structure,marketplace-social.js,api-proxy.js, Swagger UI, Dev Portal, and Chain Explorer.Closes #50