Skip to content

Start forging API#1

Open
mrv777 wants to merge 2 commits intoArdorNxtCommunity:masterfrom
mrv777:master
Open

Start forging API#1
mrv777 wants to merge 2 commits intoArdorNxtCommunity:masterfrom
mrv777:master

Conversation

@mrv777
Copy link

@mrv777 mrv777 commented Aug 2, 2018

Start adding to forging API and test editing these files correctly

mrv777 added 2 commits August 2, 2018 17:02
Start adding to forging API and test editing these files correctly
@aajajim aajajim added the enhancement New feature or request label Aug 3, 2018
@aajajim
Copy link

aajajim commented Aug 3, 2018

Hello,
Thank you for this pull request.
The issue with the build is coming from the new parameters of some call that don't have definitions in the _api_inputs.json file.
So when the engine tries to build the documentation, it goes from the getParam('param','visibility','additionnal') function and search for 'param' in the json file, and if it does not exist or does not have a type & description it fails.

I didn't add all the parameters in this file, because there a lot of them, so I was trying to add them as soon as I encounter a new parameter.

If you want to be sure that your commit will build, I would recommand you first to build it locally:
build exec middleman server then to see the rendered page (a second step of build is done here) at localhost:4567

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants