Skip to content

Conversation

@eighttrigrams
Copy link
Collaborator

@eighttrigrams eighttrigrams commented Jan 17, 2026

Same reasoning as #107.

This

Screenshot 2026-01-17 at 13 32 08

brings us closer to

Screenshot 2026-01-17 at 13 30 23

(The "Starred Repositories" could be converted to "Watched")


Open with Devin

Copilot AI review requested due to automatic review settings January 17, 2026 13:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR hides the Projects and Packages navigation links from user profile pages in Forkana by commenting out the relevant template code sections.

Changes:

  • Wrapped Projects link section in comment blocks indicating it's not used in Forkana
  • Wrapped Packages link section in comment blocks indicating it's not used in Forkana

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
templates/user/overview/header.tmpl Comments out Projects and Packages navigation links in the main template
custom/templates/user/overview/header.tmpl Comments out Projects and Packages navigation links in the custom template

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

<div class="ui small label">{{.RepoCount}}</div>
{{end}}
</a>
{{/* Projects link hidden - not used in Forkana
Copy link

Copilot AI Jan 17, 2026

Choose a reason for hiding this comment

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

Comment syntax is incorrect for Go templates. Multi-line comments should use {{/* at the start and */}} at the end on the same line, or use proper block comment format. The opening comment marker should be on its own or followed immediately by the closing marker.

Copilot uses AI. Check for mistakes.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think (and I let Claude look at this, too), that this comment is BS, but will confirm again

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I checked again, we did the exact same in PR 107 and there was no complaint. Official docs say comments may contain newlines.

Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional flags.

Open in Devin Review

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@eighttrigrams eighttrigrams force-pushed the hide-projects-packages-links branch from debfedd to 0024ad4 Compare January 17, 2026 13:31
@eighttrigrams eighttrigrams changed the title Hide Projects and Packages links from user profile Hide Projects, Packages, and Public Activity links from user profile Jan 17, 2026
@pedrogaudencio pedrogaudencio self-requested a review January 21, 2026 14:11
Copy link
Collaborator

@pedrogaudencio pedrogaudencio left a comment

Choose a reason for hiding this comment

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

Possibly to fully disable the projects, packages and activity tabs and functionalities we can add some configuration directives to custom/conf/app.ini (and a reference to that in the README.md):

[repository]
DISABLED_REPO_UNITS = repo.projects

[packages]
ENABLED = false

[explore]
ORGANIZATIONS_PAGE_DISABLED = true

[service]
ENABLE_USER_HEATMAP = false

Otherwise lgtm! 👍

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants