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

fix: amend user password migration #13662

Merged
merged 1 commit into from
Feb 13, 2024
Merged

Conversation

zomars
Copy link
Member

@zomars zomars commented Feb 13, 2024

What does this PR do?

Copy link
Contributor

github-actions bot commented Feb 13, 2024

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

@github-actions github-actions bot added the ❗️ migrations contains migration files label Feb 13, 2024
@zomars zomars changed the title Update migration.sql fix: amend user password migration Feb 13, 2024
@keithwillcode keithwillcode added api area: API, enterprise API, access token, OAuth core area: core, team members only foundation labels Feb 13, 2024
@graphite-app graphite-app bot requested a review from a team February 13, 2024 18:22
@zomars zomars enabled auto-merge (squash) February 13, 2024 18:23
Copy link

vercel bot commented Feb 13, 2024

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

Name Status Preview Comments Updated (UTC)
ai ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 13, 2024 6:25pm
1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Feb 13, 2024 6:25pm

Copy link

graphite-app bot commented Feb 13, 2024

Graphite Automations

A Graphite automation took an action on this PR • (02/13/24)

1 reviewer was added based on Keith Williams's automation, 'Add foundation team as reviewer'

Copy link
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! 🙌

Copy link

deploysentinel bot commented Feb 13, 2024

Current Playwright Test Results Summary

✅ 447 Passing - ⚠️ 26 Flaky

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

(Last updated on 02/13/2024 06:39:14pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: 5fc30e1

Started: 02/13/2024 06:30:08pm UTC

⚠️ Flakes

📄   apps/web/playwright/booking/multipleEmailQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Multiple Email Question and Each Other Question Booking With Multiple Email Question and Radio group Question Booking With Multiple Email Question and Short text question Multiple Email required and Short text required
Retry 1Initial Attempt
0% (0) 0 / 270 runs
failed over last 7 days
3.70% (10) 10 / 270 runs
flaked over last 7 days

📄   apps/web/playwright/booking/selectQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Phone Question and Each Other Question Booking With Select Question and checkbox Question Select required and checkbox required
Retry 2Retry 1Initial Attempt
0% (0) 0 / 274 runs
failed over last 7 days
3.65% (10) 10 / 274 runs
flaked over last 7 days

📄   apps/web/playwright/booking/longTextQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Long Text Question and Each Other Question Booking With Long Text Question and Phone Question Long Text and Phone required
Retry 1Initial Attempt
0% (0) 0 / 278 runs
failed over last 7 days
5.76% (16) 16 / 278 runs
flaked over last 7 days

📄   apps/web/playwright/booking/radioGroupQuestion.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Booking With Radio Question and Each Other Question Booking With Radio Question and Address Question Booking With Radio Question and checkbox Question Radio and checkbox not required
Retry 1Initial Attempt
0.36% (1) 1 / 281 run
failed over last 7 days
3.91% (11) 11 / 281 runs
flaked over last 7 days

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

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Login and logout tests -- legacy Login flow validations -- legacy Should warn when user does not exist
Retry 1Initial Attempt
0% (0) 0 / 272 runs
failed over last 7 days
4.41% (12) 12 / 272 runs
flaked over last 7 days

📄   apps/web/playwright/booking/checkboxGroupQuestion.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
Booking With Checkbox Group Question and Each Other Question Booking With Checkbox Group Question and Address Question Booking With Checkbox Group Question and checkbox Question Checkbox Group required and checkbox required
Retry 1Initial Attempt
0% (0) 0 / 282 runs
failed over last 7 days
6.38% (18) 18 / 282 runs
flaked over last 7 days
Booking With Checkbox Group Question and Each Other Question Booking With Checkbox Group Question and Address Question Booking With Checkbox Group Question and Long text Question Checkbox Group and Long text not required
Retry 1Initial Attempt
0% (0) 0 / 281 runs
failed over last 7 days
4.98% (14) 14 / 281 runs
flaked over last 7 days

📄   apps/web/playwright/booking/phoneQuestion.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
Booking With Phone Question and Each Other Question Booking With Phone Question and Address Question Booking With Phone Question and multiselect Question Phone and multiselect text required
Retry 1Initial Attempt
0% (0) 0 / 285 runs
failed over last 7 days
4.91% (14) 14 / 285 runs
flaked over last 7 days
Booking With Phone Question and Each Other Question Booking With Phone Question and Address Question Booking With Phone Question and Short text question Phone required and Short text not required
Retry 1Initial Attempt
0.35% (1) 1 / 283 run
failed over last 7 days
5.65% (16) 16 / 283 runs
flaked over last 7 days

📄   apps/web/playwright/booking/addressQuestione2e/addressQuestion.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
Booking With Address Question and Each Other Question Booking With Address Question and Checkbox Group Question Address required and checkbox group required
Retry 1Initial Attempt
0.69% (2) 2 / 290 runs
failed over last 7 days
5.86% (17) 17 / 290 runs
flaked over last 7 days
Booking With Address Question and Each Other Question Booking With Address Question and Number Question Address required and Number required
Retry 1Initial Attempt
0% (0) 0 / 286 runs
failed over last 7 days
3.85% (11) 11 / 286 runs
flaked over last 7 days

📄   apps/web/playwright/integrations-stripe.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Stripe integration Pending payment booking should not be confirmed by default
Retry 2Retry 1Initial Attempt
0.70% (2) 2 / 284 runs
failed over last 7 days
14.44% (41) 41 / 284 runs
flaked over last 7 days

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

Top 1 Common Error Messages

null

8 Test Cases Affected

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Popup Tests should open embed iframe on click - Configured with light theme
Retry 1Initial Attempt
0.69% (2) 2 / 291 runs
failed over last 7 days
52.92% (154) 154 / 291 runs
flaked over last 7 days
Popup Tests should be able to reschedule
Retry 1Initial Attempt
-114.07% (-154) -154 / 135 runs
failed over last 7 days
114.81% (155) 155 / 135 runs
flaked over last 7 days
Popup Tests should open Routing Forms embed on click
Retry 1Initial Attempt
-114.18% (-153) -153 / 134 runs
failed over last 7 days
114.18% (153) 153 / 134 runs
flaked over last 7 days
Popup Tests Floating Button Popup Pro User - Configured in App with default setting of system theme should open embed iframe according to system theme when no theme is configured through Embed API
Retry 1Initial Attempt
-114.18% (-153) -153 / 134 runs
failed over last 7 days
114.18% (153) 153 / 134 runs
flaked over last 7 days
Popup Tests Floating Button Popup Pro User - Configured in App with default setting of system theme should open embed iframe according to system theme when configured with 'auto' theme using Embed API
Retry 1Initial Attempt
-114.18% (-153) -153 / 134 runs
failed over last 7 days
114.18% (153) 153 / 134 runs
flaked over last 7 days
Popup Tests Floating Button Popup Pro User - Configured in App with default setting of system theme should open embed iframe(Booker Profile Page) with dark theme when configured with dark theme using Embed API
Retry 1Initial Attempt
-113.43% (-152) -152 / 134 runs
failed over last 7 days
113.43% (152) 152 / 134 runs
flaked over last 7 days
Popup Tests Floating Button Popup Pro User - Configured in App with default setting of system theme should open embed iframe(Event Booking Page) with dark theme when configured with dark theme using Embed API
Retry 1Initial Attempt
-113.43% (-152) -152 / 134 runs
failed over last 7 days
113.43% (152) 152 / 134 runs
flaked over last 7 days
Popup Tests prendered embed should be loaded and apply the config given to it
Retry 1Initial Attempt
-114.29% (-152) -152 / 133 runs
failed over last 7 days
114.29% (152) 152 / 133 runs
flaked over last 7 days

📄   apps/web/playwright/event-types.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Event Types tests -- legacy user enabling recurring event comes with default options
Retry 1Initial Attempt
0% (0) 0 / 290 runs
failed over last 7 days
3.79% (11) 11 / 290 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 Test preview should return correct route
Retry 1Initial Attempt
0% (0) 0 / 293 runs
failed over last 7 days
12.97% (38) 38 / 293 runs
flaked over last 7 days

📄   apps/web/playwright/organization/across-org/across-org.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
user1NotMemberOfOrg1 is part of team1MemberOfOrg1 EventTypes listing should show correct link for user events and team1MemberOfOrg1's events
Retry 1Initial Attempt
1.05% (3) 3 / 287 runs
failed over last 7 days
11.50% (33) 33 / 287 runs
flaked over last 7 days

📄   packages/embeds/embed-core/playwright/tests/inline.e2e.ts • 1 Flake

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes
Inline Iframe Inline Iframe - Configured with Dark Theme
Retry 1Initial Attempt
0.69% (2) 2 / 289 runs
failed over last 7 days
45.33% (131) 131 / 289 runs
flaked over last 7 days

📄   apps/web/playwright/profile.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
Update Profile Cannot update a users email when existing user has same email (verification enabled)
Retry 1Initial Attempt
0% (0) 0 / 178 runs
failed over last 7 days
51.69% (92) 92 / 178 runs
flaked over last 7 days
Update Profile Can update a users email (verification enabled)
Retry 1Initial Attempt
11.35% (21) 21 / 185 runs
failed over last 7 days
54.05% (100) 100 / 185 runs
flaked over last 7 days

View Detailed Build Results


Comment on lines +29 to +30
-- TODO for next migration.
-- ALTER TABLE "users" DROP COLUMN "password";
Copy link
Member Author

Choose a reason for hiding this comment

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

TODO

@zomars zomars merged commit e1f2dba into main Feb 13, 2024
38 of 39 checks passed
@zomars zomars deleted the fix/amend-password-migration branch February 13, 2024 18:40
@kart1ka
Copy link
Contributor

kart1ka commented Feb 13, 2024

HI @zomars The password column is still being dropped.
When running yarn workspace @calcom/prisma db-migrate a new migration is being created which drops the password column from users table.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api area: API, enterprise API, access token, OAuth core area: core, team members only foundation ❗️ migrations contains migration files
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants