[ELE-525] edr monitor send-report not sending the .html report

[RoiTabach]

Describe the bug
When running edr monitor send-report, we get the slack report summary to the slack channel, we have a log printout saying Sent report to Slack. but we don't have the report sent to the slack channel
To Reproduce
Steps to reproduce the behavior:

1. run some tests (so there's something to report on...)
2. run send-report. Example cmdline: edr monitor send-report -st <token> -ch roi-playground -t snowflake --days-back 1 --env prod
3. See the report summary message sent to the slack
4. see no html file in the slack.
5. see the log saying "sent report to slack"

Expected behavior
a .html file containing the report being passed in the slack channel

Screenshots
logs look like this

but slack channel looks like this

Environment (please complete the following information):

• edr Version: master
• dbt package Version: latest

Additional context
raised in support thread by Shantanu Panwar https://elementary-community.slack.com/archives/C02CTC89LAX/p1678816110540659

_{From SyncLinear.com | ELE-525}

[Maayan-s]

I'm not sure it's related, but maybe you should merge @oravi branch here before you start working on this:
#746

[RoiTabach]

Seems like this is the fix, that's the exact part of the code I thought was problematic

[RoiTabach]

This fix is not enough, local runs still not sending the report.

Debugging got me this errmsg: The server responded with: {'ok': False, 'error': 'missing_scope', 'needed': 'files:read', 'provided': 'channels:join,channels:read,chat:write,files:write,users:read,groups:read,channels:history,users:read.email'} for the file upload - wondering if this is the root cause

[Maayan-s]

Probably it! It's missing from our docs:

If the user confirms we just need to fix the docs.

[RoiTabach]

Sorry, it seems not necessarily it. I got to the above errmsg when running

self.client.files_upload_v2(channel='C04P9N971TM',initial_comment=None,file='/Users/roitabach/workspace/dbt-data-reliability/integration_tests/elementary_report.html',request_file_info=True)

But when altering request_file_info to False (per the codebase) actually gives a response of "OK"

But no file was sent

[RoiTabach]

slackapi/python-slack-sdk#1342

issue in slack sdk

fixed by pip install --update slack-sdk !

adding a requirment for slack-sdk to be above 3.20.1

[RoiTabach]

This was fixed in PR #746