Skip to content

Commit

Permalink
fix(ingest/slack): tweak docs for slack source (#10007)
Browse files Browse the repository at this point in the history
  • Loading branch information
hsheth2 authored Mar 11, 2024
1 parent 7540e64 commit bcae7ac
Showing 1 changed file with 3 additions and 11 deletions.
14 changes: 3 additions & 11 deletions metadata-ingestion/src/datahub/ingestion/source/slack/slack.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,7 @@
platform_name,
support_status,
)
from datahub.ingestion.api.source import (
SourceReport,
TestableSource,
TestConnectionReport,
)
from datahub.ingestion.api.source import Source, SourceReport
from datahub.ingestion.api.workunit import MetadataWorkUnit
from datahub.metadata.schema_classes import (
CorpUserEditableInfoClass,
Expand Down Expand Up @@ -89,10 +85,10 @@ class SlackSourceReport(SourceReport):
PLATFORM_NAME = "slack"


@platform_name(PLATFORM_NAME)
@platform_name("Slack")
@config_class(SlackSourceConfig)
@support_status(SupportStatus.TESTING)
class SlackSource(TestableSource):
class SlackSource(Source):
def __init__(self, ctx: PipelineContext, config: SlackSourceConfig):
self.ctx = ctx
self.config = config
Expand All @@ -107,10 +103,6 @@ def create(cls, config_dict, ctx):
config = SlackSourceConfig.parse_obj(config_dict)
return cls(ctx, config)

@staticmethod
def test_connection(config_dict: dict) -> TestConnectionReport:
raise NotImplementedError("This class does not implement this method")

def get_slack_client(self) -> WebClient:
return WebClient(token=self.config.bot_token.get_secret_value())

Expand Down

0 comments on commit bcae7ac

Please sign in to comment.