Skip to content
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

Documentation cleaning and minor fixes #261

Merged
merged 1 commit into from
Feb 19, 2023

Conversation

mippersiel
Copy link
Contributor

@mippersiel mippersiel commented Jan 17, 2023

  • Added 'auth_opt_' to all configuration elements
  • Removed all tabs
  • Fixed indentations in tables
  • Fixed minor errors in javascript section and added link to Go otto
  • Added example scripts in javacript section
  • Added examples in HTTP section for JSON queries

-Removed all tabs
-Fixed indentations in tables
-Fixed minor errors in javascript section and added link to Go otto
-Added example scripts in javacript section
-Added examples in HTTP section for JSON queries
@iegomez
Copy link
Owner

iegomez commented Feb 17, 2023

Sorry, I was on vacation and work's been brutal since I came back.
I'll take a look as soon as I can.

@iegomez
Copy link
Owner

iegomez commented Feb 17, 2023

He, after checking it, I'm a bit conflicted with this PR: it happens that I do prefer tabs over spaces (shit, flame wars incoming, at least one of my coworkers would want to punch me, but he's on vacation right now). 😅

@mippersiel
Copy link
Contributor Author

😆 No worries at all, I'll do the update from spaces to tabs. As for the contents, it looks good?

@iegomez
Copy link
Owner

iegomez commented Feb 17, 2023

He, it was just a joke while I was going over issues and PRs last night.
I'll take a better look today. 😉

@iegomez iegomez merged commit 9e9d90b into iegomez:master Feb 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants