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: typo in conferencing.repository.ts filename (#19161) #19173

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mohdaquib171
Copy link

What does this PR do?

This PR fixes a typo in the filename conferencing.respository.ts, changing it to conferencing.repository.ts.
All related imports have also been updated to reflect this change.

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • Documentation update not required (N/A).
  • I confirm automated tests are in place: N/A

How should this be tested?

  • Check that the file conferencing.respository.ts has been renamed to conferencing.repository.ts.
  • Verify that all imports referring to this file have been updated without any errors.

Checklist

  • I have read the contributing guide.
  • My code follows the style guidelines of this project.
  • I have checked that my changes generate no new warnings.

@mohdaquib171 mohdaquib171 requested a review from a team as a code owner February 7, 2025 14:19
Copy link

vercel bot commented Feb 7, 2025

@mohdaquib171 is attempting to deploy a commit to the cal-staging Team on Vercel.

A member of the Team first needs to authorize it.

@graphite-app graphite-app bot added the community Created by Linear-GitHub Sync label Feb 7, 2025
@graphite-app graphite-app bot requested a review from a team February 7, 2025 14:19
@CLAassistant
Copy link

CLAassistant commented Feb 7, 2025

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added 🐛 bug Something isn't working 🧹 Improvements Improvements to existing features. Mostly UX/UI labels Feb 7, 2025
Copy link

graphite-app bot commented Feb 7, 2025

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (02/07/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add community label" took an action on this PR • (02/07/25)

1 label was added to this PR based on Keith Williams's automation.

@luzpaz
Copy link
Contributor

luzpaz commented Feb 7, 2025

you neglected to rename the file itself (as described in the ticket)

@mohdaquib171
Copy link
Author

Apologies, I’ve renamed the file now as per the ticket.
Let me know if it's good to go.

Copy link
Contributor

Choose a reason for hiding this comment

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

you have changed the name of the file, do any other files use imports from this file that should be changed too?

Copy link
Author

Choose a reason for hiding this comment

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

No, all imports are already updated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working community Created by Linear-GitHub Sync 🧹 Improvements Improvements to existing features. Mostly UX/UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Typo in api/v2/**/conferencing/**/conferencing.respository.ts (should be 'repository')
4 participants