Skip to content

Comments

feat: Blog Posts (WIP)#257

Merged
whitep4nth3r merged 304 commits intonpmx-dev:feat/atproto-blog-fefrom
jonathanyeong:feat/atproto-blog-fe
Feb 6, 2026
Merged

feat: Blog Posts (WIP)#257
whitep4nth3r merged 304 commits intonpmx-dev:feat/atproto-blog-fefrom
jonathanyeong:feat/atproto-blog-fe

Conversation

@Kai-ros
Copy link
Contributor

@Kai-ros Kai-ros commented Jan 28, 2026

Overview

This PR is an omnibus of all the tasks being tackled to resolve ISSUE 176.

Changes

  • Lexicons
  • Prose-styling using unocss typography
  • anthony's markdown solution
  • Standard site push - (Currently mocking the PDS push)
  • Redesign emails templates w/npmx visual language
  • TBD...

See Also

  • Daily Project Updates - HackMD

@vercel
Copy link

vercel bot commented Jan 28, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 6, 2026 0:34am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 6, 2026 0:34am
npmx-lunaria Ignored Ignored Preview Feb 6, 2026 0:34am

Request Review

@github-actions
Copy link

github-actions bot commented Jan 28, 2026

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
lunaria/files/en-GB.json Localization changed, will be marked as complete. 🔄️
lunaria/files/en-US.json Source changed, localizations will be marked as outdated.
lunaria/files/fr-FR.json Localization changed, will be marked as complete. 🔄️
lunaria/files/it-IT.json Localization changed, will be marked as complete. 🔄️
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

@vercel
Copy link

vercel bot commented Jan 30, 2026

Deployment failed with the following error:

The provided GitHub repository does not contain the requested branch or commit reference. Please ensure the repository is not empty.

jonathanyeong added a commit to jonathanyeong/npmx.dev that referenced this pull request Feb 5, 2026
These composables will be used in this PR npmx-dev#257.
jonathanyeong added a commit to jonathanyeong/npmx.dev that referenced this pull request Feb 5, 2026
These composables will be used in this PR npmx-dev#257.

Co-authored-by: Brandon Hurrington <brandon.o.hurrington@gmail.com>
@codecov
Copy link

codecov bot commented Feb 5, 2026

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
1467 1 1466 6
View the top 1 failed test(s) by shortest run time
test/unit/a11y-component-coverage.spec.ts > a11y component test coverage > should have accessibility tests for all components (or be explicitly skipped)
Stack Traces | 0.0102s run time
AssertionError: Missing a11y tests for 6 component(s):
  - AuthorAvatar.vue
  - AuthorList.vue
  - BlogPostListCard.vue
  - BlogPostWrapper.vue
  - BlueskyComment.vue
  - BlueskyComments.vue

To fix: Add tests in test/nuxt/a11y.spec.ts or add to SKIPPED_COMPONENTS in test/unit/a11y-component-coverage.spec.ts with justification.: expected 6 to equal +0

- Expected
+ Received

- 0
+ 6

 ❯ test/unit/a11y-component-coverage.spec.ts:163:12
 ❯ node_modules/.pnpm/@voidzero-dev+vite-plus-test@0.0.0-833c515fa25cef20905a7f9affb156dfa6f151ab_@types+node_4157347c09905ce729a9a0faf64ab7ea/node_modules/@voidzero-dev/vite-plus-test/dist/@vitest/runner/index.js:145:14
 ❯ node_modules/.pnpm/@voidzero-dev+vite-plus-test@0.0.0-833c515fa25cef20905a7f9affb156dfa6f151ab_@types+node_4157347c09905ce729a9a0faf64ab7ea/node_modules/@voidzero-dev/vite-plus-test/dist/@vitest/runner/index.js:915:28
 ❯ node_modules/.pnpm/@voidzero-dev+vite-plus-test@0.0.0-833c515fa25cef20905a7f9affb156dfa6f151ab_@types+node_4157347c09905ce729a9a0faf64ab7ea/node_modules/@voidzero-dev/vite-plus-test/dist/@vitest/runner/index.js:1243:24
 ❯ runWithTimeout node_modules/.pnpm/@voidzero-dev+vite-plus-test@0.0.0-833c515fa25cef20905a7f9affb156dfa6f151ab_@types+node_4157347c09905ce729a9a0faf64ab7ea/node_modules/@voidzero-dev/vite-plus-test/dist/@vitest/runner/index.js:1209:12
 ❯ node_modules/.pnpm/@voidzero-dev+vite-plus-test@0.0.0-833c515fa25cef20905a7f9affb156dfa6f151ab_@types+node_4157347c09905ce729a9a0faf64ab7ea/node_modules/@voidzero-dev/vite-plus-test/dist/@vitest/runner/index.js:1653:42
 ❯ Traces.$ node_modules/.pnpm/@voidzero-dev+vite-plus-test@0.0.0-833c515fa25cef20905a7f9affb156dfa6f151ab_@types+node_4157347c09905ce729a9a0faf64ab7ea/node_modules/@.../dist/chunks/traces.CCmnQaNT.js:142:29
 ❯ trace node_modules/.pnpm/@voidzero-dev+vite-plus-test@0.0.0-833c515fa25cef20905a7f9affb156dfa6f151ab_@types+node_4157347c09905ce729a9a0faf64ab7ea/node_modules/@.../dist/chunks/test.B8ej_ZHS.js:239:23
 ❯ runTest node_modules/.pnpm/@voidzero-dev+vite-plus-test@0.0.0-833c515fa25cef20905a7f9affb156dfa6f151ab_@types+node_4157347c09905ce729a9a0faf64ab7ea/node_modules/@voidzero-dev/vite-plus-test/dist/@vitest/runner/index.js:1653:17

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

jonathanyeong and others added 2 commits February 5, 2026 21:35
Also fix the layout shifts caused by nested anchor tags on AuthorAvatar
@whitep4nth3r
Copy link
Contributor

Can we point this branch to the upstream feat/atproto-blog-fe and merge it there?

@whitep4nth3r whitep4nth3r changed the base branch from main to feat/atproto-blog-fe February 6, 2026 12:43
@whitep4nth3r
Copy link
Contributor

Ok I changed the base branch to be the feat/atproto-blog-fe branch for the merge.

I think we can merge this into the feature branch and then go from there.

@whitep4nth3r whitep4nth3r marked this pull request as ready for review February 6, 2026 12:46
@whitep4nth3r whitep4nth3r merged commit 71f5cca into npmx-dev:feat/atproto-blog-fe Feb 6, 2026
5 of 7 checks passed
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.

Setup blog