Skip to content

Add visual applications directory - #3

Draft
santoshkumarradha wants to merge 1 commit into
mainfrom
agent/visual-applications-directory
Draft

santoshkumarradha wants to merge 1 commit into
mainfrom
agent/visual-applications-directory

Conversation

@santoshkumarradha

Copy link
Copy Markdown
Member

What changed

  • Replaces the text-only application list with a purpose-first, three-category visual directory.
  • Adds 12 linked 512px thumbnails for the public application catalogue.
  • Keeps repository acronyms out of tile labels while retaining links to each repository.

Validation

  • Verified all 12 assets are square and referenced from the profile README.
  • Verified every tile target resolves to its intended public Agent-Field repository.
  • Reviewed the local desktop and narrow layout.

@AbirAbbas

Copy link
Copy Markdown

Two things that need a decision before this can go in, neither of them about the content itself (branch is already on top of current main, all 12 assets are present, square, referenced, and every tile target is a public non-archived repo):

The red cla check isn't about this PR. .github/workflows/cla.yml sets remote-organization-name/remote-repository-name but only passes GITHUB_TOKEN, so the action bails with "Please add a personal access token as an environment variable for writing signatures in a remote repository/organization""Could not retrieve repository contents". It has failed the same way on every PR in this repo, including #1 and #2 which were merged anyway, and there's no cla-signatures.json in the tree. It also can't be fixed from a PR branch — pull_request_target loads the workflow from main — so it needs either a PERSONAL_ACCESS_TOKEN secret or dropping the two remote-* inputs (the signatures file would then live in this same repo), on main.

This collides with #4. #4 adds a person-to-brief-af row to the exact table this PR deletes; git merge of the two heads conflicts in profile/README.md. Whichever lands second needs a manual fix-up, and the new grid has no tile for person-to-brief-af — so merging this after #4 would silently drop that entry unless a 13th thumbnail gets added.

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.

2 participants