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

Show notice when syncing #310

Merged
merged 2 commits into from
Feb 12, 2024
Merged

Show notice when syncing #310

merged 2 commits into from
Feb 12, 2024

Conversation

stage-rl
Copy link
Collaborator

No description provided.

@stage-rl
Copy link
Collaborator Author

Screenshot_20240119_103828

@stage-rl stage-rl requested a review from jsuchal January 19, 2024 09:40
@@ -13,6 +13,7 @@ def sync_all
EventBus.publish(:box_sync_all_requested)

Current.tenant.boxes.sync_all
redirect_back_or_to message_threads_path, notice: t("boxes.sync_all_running")
Copy link
Member

Choose a reason for hiding this comment

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

ked je to event :box_sync_all_requested tak aj tu by som cakal

Suggested change
redirect_back_or_to message_threads_path, notice: t("boxes.sync_all_running")
redirect_back_or_to message_threads_path, notice: t("boxes.sync_all_requested")

@@ -244,6 +244,8 @@ sk:
sender_uri: "URI odosielateľa"
recipient_uri: "URI prijímateľa"
box: "Schránka správy"
boxes:
sync_all_running: "Synchronizácia správ bola spustená a prebieha na pozadí"
Copy link
Member

Choose a reason for hiding this comment

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

Prilis technicky slovnik na moj vkus a ked je tu taketo tlacitko, tak drzime rovnaky slovnik.

image

Suggested change
sync_all_running: "Synchronizácia správ bola spustená a prebieha na pozadí"
sync_all_running: "Sťahovanie nových správ bolo spustené."

Copy link
Member

@jsuchal jsuchal left a comment

Choose a reason for hiding this comment

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

Nejake male veci

@stage-rl stage-rl requested a review from jsuchal February 9, 2024 13:58
@jsuchal jsuchal merged commit b813d4a into main Feb 12, 2024
5 of 6 checks passed
@jsuchal jsuchal deleted the GO-447/notice_when_syncing branch February 12, 2024 10:30
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.

2 participants