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: dbModel no such file or directory error #11

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

Conversation

dhairyashiil
Copy link

Hello Aditya,

The src/tours/createDBModel.ts file was causing the issue.

We were getting the error ENOENT: no such file or directory because two files were changed in Rocket.Chat:

  1. apps/meteor/server/models/raw/Messages.ts is now packages/models/src/models/Messages.ts
  2. apps/meteor/server/models/Messages.ts is no longer present. Based on the information in the step, I can see some details are now located in apps/meteor/server/models.ts so I'm currently pointing the step at this position.

Maybe we need to update the information explained in steps too.

@dhairyashiil
Copy link
Author

Some Screenshots from Tour '07 - How to create a DB model'

image

and

image

@AdityaSingh-02
Copy link
Collaborator

apps/meteor/server/models/Messages.ts is no longer present

in this case the tour will be irrelevant and misleading for new contributors, rather than that we can fix it so that it points to correct steps

@dhairyashiil
Copy link
Author

apps/meteor/server/models/Messages.ts is no longer present

in this case the tour will be irrelevant and misleading for new contributors, rather than that we can fix it so that it points to correct steps

Okay, I understand your point. I will look into it and update the PR.

…and also updated the search strings for them
@dhairyashiil
Copy link
Author

Hello @AdityaSingh-02,

I understand what caused the issue. Earlier, in the apps/meteor/server/models/ folder, there were separate files for different components interacting with the registration of the DB model. Now, Rocket.Chat has merged all that code into a single file, apps/meteor/server/models.ts.

I found that the code which was previously in apps/meteor/server/models/Messages.ts is now present in this new file, along with other code.

Essentially, I’ve broken down one large step into three smaller steps, making it easier for users to understand.
It looks good now. Have a look:
Here are the screenshots:

image

image

image

image

@Sayan4444
Copy link
Contributor

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants