fix: use latest tag line "better" -> "fast, modern"#1029
fix: use latest tag line "better" -> "fast, modern"#1029shuuji3 wants to merge 3 commits intonpmx-dev:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Lunaria Status Overview🌕 This pull request will trigger status changes. Learn moreBy default, every PR changing files present in the Lunaria configuration's You can change this by adding one of the keywords present in the Tracked Files
Warnings reference
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
📝 WalkthroughWalkthroughThis pull request updates the product tagline and description text across the application from "A better browser for the npm registry" to "A fast, modern browser for the npm registry". Changes appear in Vue components, email templates, and localisation files. No logic, control flow, or public API/signature changes were made. Suggested reviewers
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
assets/email-templates/reset-password.html (1)
154-160:⚠️ Potential issue | 🟡 MinorFix typo in the footer tagline.
The word “registryt” should be “registry”.🔧 Proposed fix
- >, a fast, modern browser for the npm registryt + >, a fast, modern browser for the npm registry
🧹 Nitpick comments (2)
i18n/locales/en.json (1)
5-5: Minor text redundancy (optional consideration).The description now contains "modern" twice: "a fast, modern browser" and "with a modern interface". This is not a blocker, but you might consider rewording the second part to avoid repetition, e.g., "with a sleek interface" or "with a clean interface".
lunaria/files/en-US.json (1)
5-5: Same text redundancy as en.json.This has the same "modern" repetition as
i18n/locales/en.json. If you decide to address the redundancy, ensure both files are updated consistently.
|
Might be a good idea to ask in the Discord for translations on this PR. |
37d8837 to
a5239e6
Compare
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
assets/email-templates/reset-password.html (1)
154-160:⚠️ Potential issue | 🟡 MinorFix typo in footer tagline (“registryt”).
Line 159 has a stray “t” at the end of “registry”.[suggested fix]
🛠️ Proposed fix
- >, a fast, modern browser for the npm registryt + >, a fast, modern browser for the npm registry
This PR replace the all previous tagline with the new one.
Before

After

TODO: