Skip to content

Conversation

@hmd-ali
Copy link
Contributor

@hmd-ali hmd-ali commented Oct 12, 2025

No description provided.

@hmd-ali hmd-ali requested a review from wiktoriavh October 12, 2025 21:38
};
};

const GUIDES_DIR = fileURLToPath(new URL('../commands/guides/subjects/', import.meta.url));
Copy link
Member

Choose a reason for hiding this comment

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

since main got merged into this branch, could you use the new function that gets the correct usl? assert url or something it was called, inside markdown.js

prod path would be './commands/guides/subjects/'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@wiktoriavh I don't see the new function, are you sure it was part of the merge? iirc the function was added and we tested prod but didn't work, so that wasn't committed and pushed.

Copy link
Member

Choose a reason for hiding this comment

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

I can't find it anymore myself. 😱 I swear there was a function for that. i will take another look.

Copy link
Member

@wiktoriavh wiktoriavh Oct 18, 2025

Choose a reason for hiding this comment

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

I found it, it's in a different open PR and mixed it up.

https://github.com/r-webdev/webdev-bot/blob/bbb9c892183548eff2cbacef0ee29c92009839df/src/util/markdown.ts

Since this is not on main, nothing you have to do then. it's for another PR.

@wiktoriavh wiktoriavh changed the title Feat/guides feat: guides Oct 18, 2025
@hmd-ali hmd-ali changed the title feat: guides feat: add /guides command and ability to sync all guides to a discord channel Oct 18, 2025
@hmd-ali hmd-ali merged commit 333e537 into main Oct 18, 2025
1 check passed
@wiktoriavh wiktoriavh deleted the feat/guides branch October 18, 2025 14:28
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