Skip to content

Conversation

@eirikhanasand
Copy link

@eirikhanasand eirikhanasand commented Dec 15, 2024

"flexsearch": "^0.7.21",
"glob": "^7.2.0",
"node-fetch": "^2.6.6",
"pm2": "^5.1.2"
Copy link
Member

Choose a reason for hiding this comment

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

Why was pm2 removed?

Copy link
Author

@eirikhanasand eirikhanasand Dec 15, 2024

Choose a reason for hiding this comment

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

Because it was installed but there is no reference to it in the code, nor is it referenced by any other dependency neither top level nor sublevel nor any script nor the dockerfile. In other words, I found no indication that it was being used. Saw no reason to include it when it isnt used.

@eirikhanasand
Copy link
Author

Added a commit for discord.js-docs v 0.4.3 which was published earlier today, where an issue in that library that caused a top level error was resolved (see TeeSeal/discord.js-docs#30). I also changed the default query "Client" to "a", and moved this to constants.ts and named it DEFAULT_SEARCH_QUERY to make it more explicit what the role of this variable is. Previous to this PR there was no default query, which resulted in no results appearing before you started typing. I feel like an alphabetically ordered list is better than just "Client" appearing, which was why I changed it to just "a".

@eirikhanasand
Copy link
Author

Added DEFAULT_SEARCH_QUERY for mdn docs as well (this was already "a", but now refers to the same constant instead of just a inline char. Also moved the DJSAutocomplete from InteractionCreateHandler.ts -> commands/docs/djs.ts to ensure both files follow the same structure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

🔧 Chore: Outdated packages and discordjs version 🐛 Bug: Application does not respond when querying MDN docs.

2 participants