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: Slack integration #2247

Draft
wants to merge 142 commits into
base: main
Choose a base branch
from
Draft

feat: Slack integration #2247

wants to merge 142 commits into from

Conversation

JanCizmar
Copy link
Contributor

No description provided.

huglx and others added 30 commits January 25, 2024 13:19
…ack-integration

# Conflicts:
#	backend/data/src/main/kotlin/io/tolgee/constants/Message.kt
…to prevent duplicate event notifications. Messages now update when translations or states change within 2 hours.
…zhosov/slack-integration

# Conflicts:
#	backend/data/src/main/kotlin/io/tolgee/constants/Message.kt
#	backend/data/src/main/kotlin/io/tolgee/model/Project.kt
#	backend/data/src/main/resources/db/changelog/schema.xml
#	webapp/src/service/apiSchema.generated.ts
…zhosov/slack-integration

# Conflicts:
#	backend/data/src/main/kotlin/io/tolgee/constants/Message.kt
#	webapp/src/service/apiSchema.generated.ts
…olgee-platform into ivanmanzhosov/slack-integration
huglx and others added 8 commits April 26, 2024 14:09
…#2284)

I have added 
- endpoint which returns data for activity in paginated form
- filter option, that allows to filter translations by provided revision
id (only keys mentioned in revision are returned)

**Why this?**
- When user executes operation modifying many columns, there is no way
in the system where you can check what specifically was changed.
- When operations are executed, users are notified in slack and can
usually can get to the platform to key edition view by clicking a button
in the Slack message. This works fine when single key is affected by the
operation. However, we need to provide similar path when larger amount
data is imported so user is redirected to the Activity view and from
there, they're able to go to translations view.

This is blocking the Slack integration, so we need simple solution
before we implement the ultimate activity grouping and notification
system.

@stepan662 can you please add frontend for the Activity view? This
should be pretty the same as it is now, we just need to add pagination
@ZuzanaOdstrcilova can you create a UI proposal of how it should look in
the translations view
- IMO, we should display this filter information super clearly, since
this is not a standard filter, which is selectable from filter menu
- We can show - the date / time of the activity, the autor, and the
activity type (e.g., key creation, translations update, import, etc)
 - Let's discuss it Monday.
@huglx Can you please sync with Stepan, so we redirect users from Slack
to the activity when import is executed?

Thanks!
Copy link

restack-app bot commented May 3, 2024

No applications have been configured for previews targeting branch: main. To do so go to restack console and configure your applications for previews.

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.

Slack integration
3 participants