Allow forum posts to link to GitHub issue or milestoneForum issues#507
Allow forum posts to link to GitHub issue or milestoneForum issues#50710done wants to merge 98 commits intoalphaonelabs:mainfrom
Conversation
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
🚨 Missing Open Issue LinkThis pull request does not reference any open GitHub issue via the Development section. PRs must be linked to an open issue using GitHub’s "Development" section (on the right sidebar). How to fix this:
This PR will be automatically closed. You can reopen it after linking an open issue. Thank you! |
|
Caution Review failedThe pull request is closed. WalkthroughThis pull request introduces new functionality for linking GitHub resources with forum topics. A new form class with optional GitHub URL fields is created in Changes
Sequence Diagram(s)sequenceDiagram
participant U as User
participant FV as ForumTopic Form/View
participant M as ForumTopic Model
participant DB as Database
U->>FV: Submit topic form with GitHub URLs (optional)
FV->>M: Validate data and create ForumTopic instance
M->>DB: Save topic record
DB-->>M: Confirm save
M-->>FV: Return created topic
FV-->>U: Redirect to topic detail page
sequenceDiagram
participant U as User
participant TD as Topic Detail View
participant M as ForumTopic Model
participant C as ForumCategory Model
participant DB as Database
U->>TD: Request topic detail page
TD->>M: Retrieve topic by ID
M->>DB: Query topic record
DB-->>M: Return topic data
TD->>M: Increment view count
TD->>C: Fetch all forum categories
C->>DB: Query categories
DB-->>C: Return category data
TD-->>U: Render topic detail page with GitHub links
Assessment against linked issues
Possibly related PRs
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (7)
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
fixes #463
I implemented this by wrapping the GitHub issue and milestone links in a flex container using Tailwind CSS utility classes, which allowed me to horizontally align the icons and text uniformly. By applying classes like "mt-4 flex items-center space-x-4" to the container and "inline-flex items-center" to each link, I ensured consistent spacing and alignment, resulting in a cleaner and more organized appearance on the forum page.
Untitled.video.-.Made.with.Clipchamp.mp4
Summary by CodeRabbit
New Features
Style