-
Notifications
You must be signed in to change notification settings - Fork 7.2k
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
fix: AvailabilitySettings atom closing override window button #15518
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 3 Ignored Deployments
|
Graphite Automations"Add consumer team as reviewer" took an action on this PR • (06/21/24)1 reviewer was added to this PR based on Keith Williams's automation. "Add platform team as reviewer" took an action on this PR • (06/21/24)1 reviewer was added to this PR based on Keith Williams's automation. |
Updated test in fix: test - I guess that the test does not load i18n files but takes the string passed to the "t" function inferring i18n value literally. |
…#15518) * fix: close override window button * fix: test
Problem
When adding date override in availability atom there is no "Close" button like in web app.
Reason
If you see file changed, then before we displayed "Close" text as
![Screenshot 2024-06-21 at 10 22 54](https://private-user-images.githubusercontent.com/42170848/341696372-32457122-565d-41d7-abaa-62aa01922fdb.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjI2MTA1MzYsIm5iZiI6MTcyMjYxMDIzNiwicGF0aCI6Ii80MjE3MDg0OC8zNDE2OTYzNzItMzI0NTcxMjItNTY1ZC00MWQ3LWFiYWEtNjJhYTAxOTIyZmRiLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MDIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODAyVDE0NTAzNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTI4MmYxY2RkNmMwZTFkYjhlNGVmZWFjNWZmYTk5ZGE1YWFmYmFlZTdhY2RlMjk1ZGI3MTAwY2Y3YTkyZWZmNGYmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.c6USmQdtKbZdQGI59NMvJFpVvuADTEwf6eeSwaxfgog)
t("Close")}
, but no such entry exists in our i18n files only "close":Solution
Change
![Screenshot 2024-06-21 at 10 26 07](https://private-user-images.githubusercontent.com/42170848/341696707-09141787-ad88-43e1-a442-0a38e1ca4f42.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjI2MTA1MzYsIm5iZiI6MTcyMjYxMDIzNiwicGF0aCI6Ii80MjE3MDg0OC8zNDE2OTY3MDctMDkxNDE3ODctYWQ4OC00M2UxLWE0NDItMGEzOGUxY2E0ZjQyLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MDIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODAyVDE0NTAzNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTVmMjY2NzE4ZmZiNzEyZDA2Y2IzODIyYThiNDY1YzE2ZWU3OTYxOGQwM2Q2ZGM5Y2ViZGNlYmEyNTBhMmViMTMmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.HVJQ07tBDfw3NuV1fWyqmG8XoO066EFDz0CE8EjvMAs)
t("Close")}
tot("close")}
and override close button is showing: