Skip to content

Conversation

drobnikj
Copy link

@drobnikj drobnikj commented Sep 26, 2025

We are almost done with creating Apify integration for windmill.
Because we want to use OAuth, I'm adding details about it.


Important

Add Apify OAuth configuration to oauth_connect.json with authorization and token URLs, scopes, and extra parameters.

  • OAuth Configuration:
    • Add apify_oauth entry to oauth_connect.json.
    • Includes auth_url, token_url, and scopes for Apify.
    • Scopes include profile and full_api_access.
    • Adds empty extra_params object.

This description was created by Ellipsis for 1ea6647. You can customize this summary. It will automatically update as commits are pushed.

Copy link
Contributor

github-actions bot commented Sep 26, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@drobnikj drobnikj marked this pull request as ready for review September 26, 2025 12:10
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Looks good to me! 👍

Reviewed everything up to 1ea6647 in 29 seconds. Click for details.
  • Reviewed 20 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 2 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. backend/oauth_connect.json:184
  • Draft comment:
    Verify Apify OAuth endpoints with current docs. Endpoints and scopes appear consistent with our other providers.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None
2. backend/oauth_connect.json:191
  • Draft comment:
    Double-check if any extra parameters are needed, or if empty object is sufficient.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_M1xCu08jqDikUQwh

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@drobnikj
Copy link
Author

drobnikj commented Oct 3, 2025

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Oct 3, 2025
@alpetric
Copy link
Collaborator

alpetric commented Oct 3, 2025

@drobnikj I sent you an email a few days ago about next steps. Not sure if you saw it

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.

2 participants