- 
                Notifications
    
You must be signed in to change notification settings  - Fork 1.1k
 
validate api script (#10217) #10220
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
validate api script (#10217) #10220
Conversation
| 
           @ReynardoEW are we sending the json file in the discord? as discord message will be truncated when it's very big? everything else looks good.  | 
    
          
 it send summaries not full json file something like this  | 
    
| 
           @ReynardoEW it will only send the alert if there is failure right? imo json file would be better for debugging, but we can add it later if needed for jenkin jobs, or we can add link of file, as i believe filename will always same we can later use static link, but fine for now.  | 
    
          
 
 i have updated it to only send when error, yes i also think sending json link is better  | 
    
| 
           @ReynardoEW can we also automate the generation of this llms.txt file, it uses only pro-apis request/response information from the defillama-pro-openapi.json, everything else will stay the same  | 
    
          
 i think it should be possible but llms.txt file has to have description regarding the API stuff right? i think it's probably best to use stuff like https://github.com/mendableai/create-llmstxt-py and let it create it for us?  | 
    
| 
           @ReynardoEW @treeoflife2 mostly good, add a minor fix here where beta test is skipped if url is not passed: d8c2e31  | 
    
uses api-docs schemas, some error caused by wrong schemas definition so need to make changes there #10217
.env setting
BASE_API_URL=https://api.llama.fi
BETA_API_URL=
DISCORD_WEBHOOK_URL=
PRO_API_KEY=