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

Regression test fails - Kim needs access to other calendars for better coverage #1476

Open
sahalali opened this issue Dec 2, 2024 · 8 comments
Assignees

Comments

@sahalali
Copy link
Contributor

sahalali commented Dec 2, 2024

Added @kmdvs's comment from #1453

The RT aren't designed for other calendars than Tout Culture... At least, not yet, since I never tested another calendar... It would be great to give my 4 accounts access to all calendars, so I can actually make appropriate changes based on what calendar IS tested. In theory, since the code-base for all calendars is pretty similar (correct me if I am wrong), and the RT use the UI, and the UI don't change that much between calendars, nothing suggests a specific calendar should pose a particular problem other than small exceptions (like clicking on an additional button to edit something, or using an additional drop-down before saving an event).

I would need specific information on what fails to investigate, and more importantly, access to all calendars for my 4 accounts in production :)

@sahalali
Copy link
Contributor Author

sahalali commented Dec 2, 2024

@kmdvs Can you please remind me which all addresses needs to added to other calendars?

@dev-aravind
Copy link
Contributor

@sahalali @kmdvs I am updating the regression tests to run on the creation of a pull request rather than on a deployment.

@kmdvs
Copy link
Collaborator

kmdvs commented Dec 4, 2024

@sahalali the 4 accounts are
[email protected] (admin-fr)
[email protected] (guest-fr)
[email protected] (admin-en)
[email protected] (guest-en)

@kmdvs
Copy link
Collaborator

kmdvs commented Dec 16, 2024

@sahalali I still can't change to any other calendar, in Staging or in Production

@troughc troughc changed the title Regression test failing Regression test fails - Kim needs access to other calendars for better coverage Dec 18, 2024
@troughc
Copy link
Contributor

troughc commented Dec 18, 2024

@sahalali can you please add Kim's 4 emails (above) to all calendars so that regression tests work whenever you test (either from staging or production)
Right now the emails are only in Tout Culture and thus, only test Tout Culture.
The goal is that Kim's emails will be added to all calendars and all future calendars.
After you add them, please assign this to Kim so he can test.
cc @kmdvs

@sahalali
Copy link
Contributor Author

@kmdvs I've invited you to the Signe Laval calendar. All four of your accounts will receive the invitation soon.
cc @troughc

@sahalali
Copy link
Contributor Author

@sahalali To update the script that copies data from production to staging, Kim's account needs to be added to all calendars in staging. Additionally, a script should be developed to copy data locally.

@sahalali
Copy link
Contributor Author

The script that add developer accounts are now modified to add Kim's 4 accounts to staging when ever data is copied from production to staging.

From now on Kim's emails have access to all the calendars in staging.

@sahalali sahalali assigned kmdvs and unassigned sahalali Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants