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

feat(providers): add Notion #3112

Closed

Conversation

thejollyrogers
Copy link

@thejollyrogers thejollyrogers commented Jun 8, 2022

Submitting Pull Requests

General

  • Make sure you use semantic commit messages.
    Examples: "fix(google): Fixed foobar bug", "feat(accounts): Added foobar feature".
  • All Python code must formatted using Black, and clean from pep8 and isort issues.
  • JavaScript code should adhere to StandardJS.
  • If your changes are significant, please update ChangeLog.rst.
  • If your change is substantial, feel free to add yourself to AUTHORS.

Provider Specifics

In case you add a new provider:

  • Make sure unit tests are available.
  • Add an entry of your provider in test_settings.py::INSTALLED_APPS and docs/installation.rst::INSTALLED_APPS.
  • Add documentation to docs/providers.rst.
  • Add an entry to the list of supported providers over at docs/overview.rst.

@@ -120,6 +120,7 @@ Morgante Pell
Nariman Gharib
Nathan Strobbe
Niklas A Emanuelsson
NotionPacks.com
Copy link
Owner

Choose a reason for hiding this comment

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

Can you please leave this out, or, replace it with the actual author name(s) ?

"""
user_id = self.get_user(data).get("id")
workspace_id = data.get("workspace_id")
return f"user-{user_id}_workspace-{workspace_id}"
Copy link
Owner

Choose a reason for hiding this comment

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

f-strings break the test suite as it still includes python 3.5

@pennersr
Copy link
Owner

pennersr commented Aug 4, 2023

Closing -- see #3316

@pennersr pennersr closed this Aug 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants