Skip to content

fix: Fix missing reschedule_reason translation#9656

Merged
CarinaWolli merged 1 commit into
mainfrom
fix-reschedule-reason-i18n
Jun 20, 2023
Merged

fix: Fix missing reschedule_reason translation#9656
CarinaWolli merged 1 commit into
mainfrom
fix-reschedule-reason-i18n

Conversation

@hariombalhara
Copy link
Copy Markdown
Member

@hariombalhara hariombalhara commented Jun 20, 2023

What does this PR do?

Initially reported here https://threads.com/34482419960?s=g5bJ7c7Qc1URWWSNw5tnP1

fixes #9653

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • Go and do a reschedule

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Checklist

  • I haven't added tests that prove my fix is effective or that my feature works

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 20, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
api ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 20, 2023 8:03am
cal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 20, 2023 8:03am
ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 20, 2023 8:03am
web-staging ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 20, 2023 8:03am

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 20, 2023

Thank you for following the naming conventions! 🙏

"kbar_search_placeholder": "Type a command or search...",
"invalid_credential": "Oh no! Looks like permission expired or was revoked. Please reinstall again.",
"reschedule_reason_success_page": "Reschedule reason",
"reschedule_reason": "Reschedule reason",
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The reason behind the issue was missing reschedule_reason key

@hariombalhara hariombalhara changed the title reschedule_reason is used by Booking Form fix: Fix missing reschedule_reason translation Jun 20, 2023
@hariombalhara hariombalhara requested review from a team June 20, 2023 07:57
@hariombalhara hariombalhara marked this pull request as ready for review June 20, 2023 07:57
@github-actions
Copy link
Copy Markdown
Contributor

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@deploysentinel
Copy link
Copy Markdown

deploysentinel Bot commented Jun 20, 2023

Current Playwright Test Results Summary

✅ 114 Passing - ⚠️ 5 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 06/20/2023 08:07:07am UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: b6da43e

Started: 06/20/2023 08:02:54am UTC

⚠️ Flakes

📄   packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 2 Flakes

Top 1 Common Error Messages

null

2 Test Cases Affected

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests should be able to reschedule
Retry 1Initial Attempt
9.19% (17) 17 / 185 runs
failed over last 7 days
87.03% (161) 161 / 185 runs
flaked over last 7 days
Popup Tests should open embed iframe on click - Configured with light theme
Retry 1Initial Attempt
8.11% (15) 15 / 185 runs
failed over last 7 days
7.03% (13) 13 / 185 runs
flaked over last 7 days

📄   packages/app-store/routing-forms/playwright/tests/basic.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Routing Forms Seeded Routing Form Routing Link - Reporting and CSV Download
Retry 2Retry 1Initial Attempt
0.53% (1) 1 / 188 run
failed over last 7 days
12.23% (23) 23 / 188 runs
flaked over last 7 days

📄   apps/web/playwright/change-password.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Change Password Test change password
Retry 1Initial Attempt
0% (0) 0 / 303 runs
failed over last 7 days
1.98% (6) 6 / 303 runs
flaked over last 7 days

📄   apps/web/playwright/login.2fa.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
2FA Tests should allow a user to enable 2FA and login using 2FA
Retry 1Initial Attempt
0.67% (2) 2 / 297 runs
failed over last 7 days
13.80% (41) 41 / 297 runs
flaked over last 7 days

View Detailed Build Results


@CarinaWolli CarinaWolli merged commit a380087 into main Jun 20, 2023
@CarinaWolli CarinaWolli deleted the fix-reschedule-reason-i18n branch June 20, 2023 08:39
@PeerRich PeerRich added the core area: core, team members only label Jul 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only

Projects

No open projects
Status: No status

Development

Successfully merging this pull request may close these issues.

[CAL-2003] translation key for reschedule_reason missing

3 participants