-
-
Notifications
You must be signed in to change notification settings - Fork 690
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
graphql: Include the GraphiQL IDE with the add-on [WIP] #4269
base: main
Are you sure you want to change the base?
Conversation
addOns/graphql/src/main/javahelp/org/zaproxy/addon/graphql/resources/help/contents/graphql.html
Outdated
Show resolved
Hide resolved
addOns/graphql/src/main/javahelp/org/zaproxy/addon/graphql/resources/help/contents/graphql.html
Outdated
Show resolved
Hide resolved
addOns/graphql/src/main/javahelp/org/zaproxy/addon/graphql/resources/help/contents/graphql.html
Show resolved
Hide resolved
Now has a conflict too. |
buildSrc/src/main/java/org/zaproxy/gradle/tasks/GenerateGraphiQlHtmlPage.java
Outdated
Show resolved
Hide resolved
0c882bd
to
0e7d859
Compare
Signed-off-by: ricekot <github@ricekot.com>
0e7d859
to
ac2cf34
Compare
This is ready for reviews now. However, it needs zaproxy/gradle-plugin-add-on#61 (and a release of version |
Signed-off-by: ricekot <github@ricekot.com>
ac2cf34
to
9142bef
Compare
The version 0.9.0 is now released. |
} | ||
} | ||
var buttonCount = buttonPanel.getComponentCount(); | ||
buttonPanel.add(getApiExploreButton(), buttonCount - 1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should remove on unload()
.
Note I didn't review everything, spotted that while checking the core changes. |
Now has conflicts. |
No description provided.