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

v1.4.6 #4330

Merged
merged 5 commits into from
May 9, 2024
Merged

v1.4.6 #4330

merged 5 commits into from
May 9, 2024

Conversation

matiasb
Copy link
Contributor

@matiasb matiasb commented May 9, 2024

No description provided.

matiasb and others added 5 commits May 8, 2024 17:27
Related to some API latency issues we were noticing.
# What this PR does
Handle empty values in convertFiltersToBackendFormat

## Which issue(s) this PR closes
Broken Insights page


## Checklist

- [ ] Unit, integration, and e2e (if applicable) tests updated
- [x] Documentation added (or `pr:no public docs` PR label added if not
required)
- [x] Added the relevant release notes label (see labels prefixed w/
`release:`). These labels dictate how your PR will
    show up in the autogenerated release notes.
# What this PR does

It updates the slack deep link url to respect the [correct
format](https://api.slack.com/reference/deep-linking#app_or_bot)
requested [here](#4122)

## Checklist

- [x] Unit, integration, and e2e (if applicable) tests updated
- [x] Documentation added (or `pr:no public docs` PR label added if not
required)
- [x] Added the relevant release notes label (see labels prefixed w/
`release:`). These labels dictate how your PR will
    show up in the autogenerated release notes.

---------

Co-authored-by: Ildar Iskhakov <[email protected]>
Do not keep retrying on HttpErrors (eg. 403). Also, we will re-queued
periodically later.
)

Do not retry updating a message if Slack returns `cant_update_message`
API [error](https://api.slack.com/methods/chat.update#errors) (meaning
bot user has no permission to update the message).
@matiasb matiasb requested review from a team May 9, 2024 17:03
@matiasb matiasb temporarily deployed to github-pages May 9, 2024 17:04 — with GitHub Actions Inactive
@matiasb matiasb merged commit f2be571 into main May 9, 2024
52 of 53 checks passed
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.

3 participants