-
Notifications
You must be signed in to change notification settings - Fork 4
refactor: shift some text messages to work on blabsy #199
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
WalkthroughThis update implements a comprehensive rebranding of the "blabsy" platform, replacing all references to "Twitter," "tweets," and "retweets" with "Blabsy," "blabs," and "reblabs" across documentation, UI text, and code comments. Several navigation links and sidebar features are commented out, and new plugins for message formatting and message reference matching are introduced. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant UI
participant Backend
participant Plugin
User->>UI: Interacts (e.g., posts a blab, clicks "Blab" button)
UI->>Backend: Sends request (e.g., create blab, update stats)
Backend->>Plugin: (If localization needed) Load/Save messages per language
Plugin-->>Backend: Returns localized messages
Backend-->>UI: Returns response (updated data, localized UI)
UI-->>User: Displays updated UI with "Blab", "Reblab", etc.
Poem
✨ 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:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. 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 (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 20
🔭 Outside diff range comments (1)
platforms/blabsy/project.inlang/cache/plugins/ygx0uiahq6uw (1)
1-17
: 💡 Verification agent🧩 Analysis chain
Cache files should not be committed to version control
This appears to be a generated/cached plugin file that should not be committed to the repository. Cache directories and their contents should typically be added to
.gitignore
to prevent them from being tracked by version control.Additionally, the minified code makes it impossible to review for correctness, security issues, or alignment with the PR objectives.
Run the following script to check if this cache directory is properly ignored:
🏁 Script executed:
#!/bin/bash # Description: Check if the cache directory is in .gitignore # Check if .gitignore exists and contains the cache path if [ -f .gitignore ]; then echo "=== Checking .gitignore for cache patterns ===" rg "cache|\.inlang/cache" .gitignore || echo "No cache patterns found in .gitignore" else echo ".gitignore file not found" fi # Check for other cache files in the repository echo -e "\n=== Checking for other cache files ===" fd -t f . "platforms/blabsy/project.inlang/cache" | head -20Length of output: 523
🏁 Script executed:
#!/bin/bash # Description: Verify if these are generated files by checking for source files # Look for potential source files for these plugins echo "=== Searching for plugin source files ===" fd -t f -e js -e ts "mFunctionMatcher|m-function-matcher|inlang.*matcher" --exclude "cache" | head -10 # Check if there's a build process that generates these files echo -e "\n=== Checking for build scripts ===" fd "package.json" -x grep -l "build\|compile\|bundle" {} \; | head -5Length of output: 533
Remove generated cache files and ignore the cache directory
These
platforms/blabsy/project.inlang/cache/plugins/*
files are runtime-generated and shouldn’t be committed. Please:
- Remove these tracked cache files:
platforms/blabsy/project.inlang/cache/plugins/2sy648wh9sugi
platforms/blabsy/project.inlang/cache/plugins/ygx0uiahq6uw
- Add the cache directory (e.g.
platforms/blabsy/project.inlang/cache/
) to your.gitignore
so it isn’t tracked in the future.This will keep the repo clean and avoid committing minified/generated code.
♻️ Duplicate comments (1)
platforms/blabsy/project.inlang/cache/plugins/2sy648wh9sugi (1)
1-2
: Same issue: Cache files in version controlThis file has the same issues as the previous cache file - it should not be committed to version control.
🧹 Nitpick comments (12)
platforms/blabsy/src/components/aside/aside.tsx (2)
2-2
: Remove unused import ofSearchBar
.
TheSearchBar
component is commented out and no longer used here—removing its import will clean up dead code.
17-17
: Consider replacing commented-out code with a feature flag or removal.
Instead of leaving<SearchBar />
commented, use a feature toggle or remove the block to improve readability and maintainability.platforms/blabsy/src/components/modal/tweet-stats-modal.tsx (1)
24-24
: Explicitly handlenull
statsType
.
The fallback (statsType !== 'likes'
) also coversnull
; consider handlingnull
explicitly (e.g., default text) to avoid misleading titles.platforms/blabsy/src/components/user/user-nav.tsx (1)
2-2
: Consider using the project's standardcn
utility function.The import uses
cn
from 'clsx', but the codebase has a more sophisticatedcn
function ininfrastructure/eid-wallet/src/lib/utils/mergeClasses.ts
that combinesclsx
withtwMerge
for proper Tailwind CSS class conflict resolution.-import cn from 'clsx'; +import { cn } from '@lib/utils/mergeClasses';platforms/blabsy/src/components/login/login-footer.tsx (1)
1-21
: Remove dead code: clean up commented footer links
Since the footer navigation links are no longer needed, remove the commented-outfooterLinks
array and its mapping block to avoid clutter.platforms/blabsy/src/components/input/input-form.tsx (1)
94-94
: Consider improving assignment clarity in expression.The assignment within the setTimeout expression could be made clearer for better readability.
-setTimeout(() => (sidebar.style.opacity = ''), 200); +setTimeout(() => { + sidebar.style.opacity = ''; +}, 200);🧰 Tools
🪛 Biome (1.9.4)
[error] 94-94: The assignment should not be in an expression.
The use of assignments in expressions is confusing.
Expressions are often considered as side-effect free.(lint/suspicious/noAssignInExpressions)
platforms/blabsy/functions/src/normalize-stats.ts (2)
19-30
: Consider using for...of loop instead of forEach for better performanceAs flagged by static analysis,
forEach
can lead to performance issues with large arrays. Consider replacing with afor...of
loop:- usersStatsToDelete.forEach((userId) => { - functions.logger.info(`Deleting stats from ${userId}`); - - const userStatsRef = firestore() - .doc(`users/${userId}/stats/stats`) - .withConverter(tweetConverter); - - batch.update(userStatsRef, { - tweets: firestore.FieldValue.arrayRemove(tweetId), - likes: firestore.FieldValue.arrayRemove(tweetId), - }); - }); + for (const userId of usersStatsToDelete) { + functions.logger.info(`Deleting stats from ${userId}`); + + const userStatsRef = firestore() + .doc(`users/${userId}/stats/stats`) + .withConverter(tweetConverter); + + batch.update(userStatsRef, { + tweets: firestore.FieldValue.arrayRemove(tweetId), + likes: firestore.FieldValue.arrayRemove(tweetId), + }); + }🧰 Tools
🪛 Biome (1.9.4)
[error] 19-30: Prefer for...of instead of forEach.
forEach may lead to performance issues when working with large arrays. When combined with functions like filter or map, this causes multiple iterations over the same type.
(lint/complexity/noForEach)
41-44
: Consider using for...of loop for bookmark deletionSimilar to the previous forEach, this can be optimized:
- docsSnap.docs.forEach(({ id, ref }) => { - functions.logger.info(`Deleting bookmark ${id}`); - batch.delete(ref); - }); + for (const { id, ref } of docsSnap.docs) { + functions.logger.info(`Deleting bookmark ${id}`); + batch.delete(ref); + }🧰 Tools
🪛 Biome (1.9.4)
[error] 41-44: Prefer for...of instead of forEach.
forEach may lead to performance issues when working with large arrays. When combined with functions like filter or map, this causes multiple iterations over the same type.
(lint/complexity/noForEach)
platforms/blabsy/README.md (2)
20-20
: Fix word repetition in documentation.The phrase "their and other" contains a repeated word that should be corrected for better readability.
Apply this diff to improve the sentence:
- Users can see their and other followers and the following list + Users can see their followers and other users' followers and following lists🧰 Tools
🪛 LanguageTool
[duplication] ~20-~20: Possible typo: you repeated a word.
Context: ...- Users can follow and unfollow other users - Users can see their and other followers and t...(ENGLISH_WORD_REPEAT_RULE)
118-118
: Consider formatting the bare URL.The bare URL should be properly formatted as a markdown link for better documentation standards.
Apply this diff to format the URL:
-https://console.firebase.google.com/u/0/project/your-project-id/firestore/indexes +[https://console.firebase.google.com/u/0/project/your-project-id/firestore/indexes](https://console.firebase.google.com/u/0/project/your-project-id/firestore/indexes)🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
118-118: Bare URL used
null(MD034, no-bare-urls)
platforms/blabsy/src/components/layout/common-layout.tsx (1)
60-61
: Remove unnecessary empty div.Since AsideTrends is being disabled, the empty div replacement serves no purpose and should be removed entirely.
Apply this diff to clean up the PeopleLayout:
<Aside> {/* <AsideTrends /> */} - <div /> </Aside>
platforms/blabsy/src/components/input/input.tsx (1)
169-169
: Consider using for...of for better performance.Static analysis suggests replacing
forEach
withfor...of
for better performance with large arrays.Apply this refactor:
- const cleanImage = (): void => { - imagesPreview.forEach(({ src }) => URL.revokeObjectURL(src)); + const cleanImage = (): void => { + for (const { src } of imagesPreview) { + URL.revokeObjectURL(src); + }🧰 Tools
🪛 Biome (1.9.4)
[error] 169-169: Prefer for...of instead of forEach.
forEach may lead to performance issues when working with large arrays. When combined with functions like filter or map, this causes multiple iterations over the same type.
(lint/complexity/noForEach)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (42)
platforms/blabsy/README.md
(1 hunks)platforms/blabsy/functions/src/normalize-stats.ts
(1 hunks)platforms/blabsy/functions/src/notify-email.ts
(1 hunks)platforms/blabsy/project.inlang/cache/plugins/2sy648wh9sugi
(1 hunks)platforms/blabsy/project.inlang/cache/plugins/ygx0uiahq6uw
(1 hunks)platforms/blabsy/src/components/aside/aside-footer.tsx
(2 hunks)platforms/blabsy/src/components/aside/aside.tsx
(1 hunks)platforms/blabsy/src/components/aside/search-bar.tsx
(1 hunks)platforms/blabsy/src/components/common/app-head.tsx
(1 hunks)platforms/blabsy/src/components/home/update-username.tsx
(1 hunks)platforms/blabsy/src/components/input/input-form.tsx
(1 hunks)platforms/blabsy/src/components/input/input-options.tsx
(1 hunks)platforms/blabsy/src/components/input/input.tsx
(2 hunks)platforms/blabsy/src/components/layout/common-layout.tsx
(1 hunks)platforms/blabsy/src/components/layout/user-data-layout.tsx
(1 hunks)platforms/blabsy/src/components/login/login-footer.tsx
(2 hunks)platforms/blabsy/src/components/login/login-main.tsx
(1 hunks)platforms/blabsy/src/components/modal/display-modal.tsx
(1 hunks)platforms/blabsy/src/components/modal/mobile-sidebar-modal.tsx
(1 hunks)platforms/blabsy/src/components/modal/tweet-stats-modal.tsx
(1 hunks)platforms/blabsy/src/components/sidebar/more-settings.tsx
(2 hunks)platforms/blabsy/src/components/sidebar/sidebar.tsx
(1 hunks)platforms/blabsy/src/components/tweet/tweet-actions.tsx
(2 hunks)platforms/blabsy/src/components/tweet/tweet-date.tsx
(1 hunks)platforms/blabsy/src/components/tweet/tweet-share.tsx
(1 hunks)platforms/blabsy/src/components/tweet/tweet-stats.tsx
(1 hunks)platforms/blabsy/src/components/tweet/tweet.tsx
(1 hunks)platforms/blabsy/src/components/ui/follow-button.tsx
(1 hunks)platforms/blabsy/src/components/user/user-cards.tsx
(1 hunks)platforms/blabsy/src/components/user/user-follow.tsx
(1 hunks)platforms/blabsy/src/components/user/user-nav.tsx
(1 hunks)platforms/blabsy/src/components/view/view-tweet-stats.tsx
(1 hunks)platforms/blabsy/src/pages/404.tsx
(1 hunks)platforms/blabsy/src/pages/bookmarks.tsx
(2 hunks)platforms/blabsy/src/pages/home.tsx
(1 hunks)platforms/blabsy/src/pages/index.tsx
(1 hunks)platforms/blabsy/src/pages/people.tsx
(2 hunks)platforms/blabsy/src/pages/trends.tsx
(2 hunks)platforms/blabsy/src/pages/tweet/[id].tsx
(1 hunks)platforms/blabsy/src/pages/user/[id]/likes.tsx
(1 hunks)platforms/blabsy/src/pages/user/[id]/media.tsx
(1 hunks)platforms/blabsy/src/pages/user/[id]/with_replies.tsx
(1 hunks)
🧰 Additional context used
🧬 Code Graph Analysis (16)
platforms/blabsy/src/components/aside/aside.tsx (1)
platforms/blabsy/src/components/aside/aside-footer.tsx (1)
AsideFooter
(12-34)
platforms/blabsy/src/components/modal/tweet-stats-modal.tsx (1)
platforms/blabsy/src/components/view/view-tweet-stats.tsx (1)
StatsType
(22-22)
platforms/blabsy/src/components/user/user-nav.tsx (2)
infrastructure/eid-wallet/src/lib/utils/mergeClasses.ts (1)
cn
(4-6)platforms/blabsy/src/components/input/input.tsx (1)
variants
(38-41)
platforms/blabsy/src/components/user/user-follow.tsx (1)
platforms/blabsy/src/components/user/user-cards.tsx (1)
UserCards
(49-91)
platforms/blabsy/functions/src/notify-email.ts (1)
platforms/blabsy/src/components/tweet/tweet.tsx (1)
Tweet
(36-232)
platforms/blabsy/src/pages/user/[id]/likes.tsx (3)
platforms/blabsy/src/components/tweet/tweet.tsx (1)
Tweet
(36-232)platforms/blabsy/src/components/layout/common-layout.tsx (2)
ProtectedLayout
(12-18)UserLayout
(32-42)platforms/blabsy/src/components/layout/user-data-layout.tsx (1)
UserDataLayout
(12-36)
platforms/blabsy/src/components/tweet/tweet-date.tsx (1)
infrastructure/eid-wallet/src/lib/utils/mergeClasses.ts (1)
cn
(4-6)
platforms/blabsy/src/components/input/input-options.tsx (1)
platforms/blabsy/src/components/input/input.tsx (1)
variants
(38-41)
platforms/blabsy/src/components/aside/search-bar.tsx (1)
infrastructure/eid-wallet/src/lib/utils/mergeClasses.ts (1)
cn
(4-6)
platforms/blabsy/src/components/modal/mobile-sidebar-modal.tsx (1)
platforms/blabsy/src/components/modal/display-modal.tsx (1)
DisplayModal
(27-106)
platforms/blabsy/src/pages/people.tsx (1)
platforms/blabsy/src/components/layout/common-layout.tsx (2)
ProtectedLayout
(12-18)PeopleLayout
(55-65)
platforms/blabsy/src/components/tweet/tweet-share.tsx (2)
infrastructure/eid-wallet/src/lib/utils/mergeClasses.ts (1)
cn
(4-6)platforms/blabsy/src/components/tweet/tweet-actions.tsx (1)
variants
(30-38)
platforms/blabsy/src/components/layout/common-layout.tsx (1)
platforms/blabsy/src/components/aside/aside.tsx (1)
Aside
(10-22)
platforms/blabsy/src/components/sidebar/more-settings.tsx (2)
platforms/blabsy/src/components/modal/display-modal.tsx (1)
DisplayModal
(27-106)infrastructure/eid-wallet/src/lib/utils/mergeClasses.ts (1)
cn
(4-6)
platforms/blabsy/functions/src/normalize-stats.ts (1)
platforms/blabsy/src/components/tweet/tweet.tsx (1)
Tweet
(36-232)
platforms/blabsy/src/pages/trends.tsx (2)
platforms/blabsy/src/pages/bookmarks.tsx (1)
Bookmarks
(29-121)platforms/blabsy/src/components/layout/common-layout.tsx (2)
ProtectedLayout
(12-18)TrendsLayout
(44-53)
🪛 Biome (1.9.4)
platforms/blabsy/src/components/view/view-tweet-stats.tsx
[error] 92-104: Provide an explicit type prop for the button element.
The default type of a button is submit, which causes the submission of a form when placed inside a form
element. This is likely not the behaviour that you want inside a React application.
Allowed button types are: submit, button or reset
(lint/a11y/useButtonType)
platforms/blabsy/src/components/tweet/tweet-date.tsx
[error] 22-28: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
platforms/blabsy/src/components/sidebar/sidebar.tsx
[error] 91-95: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
platforms/blabsy/src/components/tweet/tweet-stats.tsx
[error] 52-52: This hook does not specify all of its dependencies: currentReplies
This dependency is not specified in the hook dependency list.
(lint/correctness/useExhaustiveDependencies)
[error] 57-57: This hook does not specify all of its dependencies: currentLikes
This dependency is not specified in the hook dependency list.
(lint/correctness/useExhaustiveDependencies)
[error] 62-62: This hook does not specify all of its dependencies: currentTweets
This dependency is not specified in the hook dependency list.
(lint/correctness/useExhaustiveDependencies)
platforms/blabsy/src/components/input/input-form.tsx
[error] 94-94: The assignment should not be in an expression.
The use of assignments in expressions is confusing.
Expressions are often considered as side-effect free.
(lint/suspicious/noAssignInExpressions)
platforms/blabsy/src/components/tweet/tweet-actions.tsx
[error] 157-157: This hook does not specify all of its dependencies: tweetIsPinned
This dependency is not specified in the hook dependency list.
(lint/correctness/useExhaustiveDependencies)
[error] 157-157: This hook specifies more dependencies than necessary: pinOpen
This dependency can be removed from the list.
(lint/correctness/useExhaustiveDependencies)
platforms/blabsy/src/components/modal/mobile-sidebar-modal.tsx
[error] 140-140: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
[error] 178-182: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
platforms/blabsy/src/components/tweet/tweet.tsx
[error] 103-103: Use a button element instead of an a element.
Anchor elements should only be used for default sections or page navigation
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
[error] 117-117: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
[error] 192-192: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
[error] 36-36: Shouldn't redeclare 'Tweet'. Consider to delete it or rename it.
'Tweet' is defined here:
(lint/suspicious/noRedeclare)
platforms/blabsy/src/components/tweet/tweet-share.tsx
[error] 42-42: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
platforms/blabsy/src/components/input/input.tsx
[error] 119-119: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
[error] 169-169: Prefer for...of instead of forEach.
forEach may lead to performance issues when working with large arrays. When combined with functions like filter or map, this causes multiple iterations over the same type.
(lint/complexity/noForEach)
[error] 228-228: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
[error] 66-66: This hook does not specify all of its dependencies: modal
This dependency is not specified in the hook dependency list.
Either include it or remove the dependency array
(lint/correctness/useExhaustiveDependencies)
platforms/blabsy/functions/src/normalize-stats.ts
[error] 19-30: Prefer for...of instead of forEach.
forEach may lead to performance issues when working with large arrays. When combined with functions like filter or map, this causes multiple iterations over the same type.
(lint/complexity/noForEach)
[error] 41-44: Prefer for...of instead of forEach.
forEach may lead to performance issues when working with large arrays. When combined with functions like filter or map, this causes multiple iterations over the same type.
(lint/complexity/noForEach)
platforms/blabsy/src/components/home/update-username.tsx
[error] 30-30: This hook does not specify all of its dependencies: errorMessage
This dependency is not specified in the hook dependency list.
(lint/correctness/useExhaustiveDependencies)
[error] 30-30: This hook does not specify all of its dependencies: user?.username
This dependency is not specified in the hook dependency list.
(lint/correctness/useExhaustiveDependencies)
[error] 30-30: This hook does not specify all of its dependencies: visited
This dependency is not specified in the hook dependency list.
This dependency is not specified in the hook dependency list.
(lint/correctness/useExhaustiveDependencies)
[error] 61-61: This hook does not specify all of its dependencies: openModal
This dependency is not specified in the hook dependency list.
Either include it or remove the dependency array
(lint/correctness/useExhaustiveDependencies)
[error] 61-61: This hook does not specify all of its dependencies: user?.updatedAt
This dependency is not specified in the hook dependency list.
Either include it or remove the dependency array
(lint/correctness/useExhaustiveDependencies)
🪛 LanguageTool
platforms/blabsy/README.md
[duplication] ~20-~20: Possible typo: you repeated a word.
Context: ...- Users can follow and unfollow other users - Users can see their and other followers and t...
(ENGLISH_WORD_REPEAT_RULE)
[style] ~123-~123: You have already used this phrasing in nearby sentences. Consider replacing it to add variety to your writing.
Context: ...esn't make this API paid 😅. - If you want to make the user stats synced with the del...
(REP_WANT_TO_VB)
🪛 markdownlint-cli2 (0.17.2)
platforms/blabsy/README.md
118-118: Bare URL used
null
(MD034, no-bare-urls)
🔇 Additional comments (61)
platforms/blabsy/functions/src/notify-email.ts (2)
29-31
: LGTM: Email subject updated correctly for Blabsy branding.The email subject change from "New Tweet" to "New Blab" aligns perfectly with the rebranding effort.
37-37
: LGTM: Email body text updated correctly.The link text change from "Link to Tweet" to "Link to Blab" maintains consistency with the rebranding.
platforms/blabsy/src/components/sidebar/more-settings.tsx (1)
14-21
: LGTM: Code formatting improvements.The formatting and indentation changes improve code readability while preserving all functional logic and component behavior.
Also applies to: 24-115
platforms/blabsy/src/pages/index.tsx (1)
11-11
: LGTM: SEO title updated for Blabsy branding.The page title change from "Twitter" to "Blabsy" correctly reflects the platform rebranding and maintains the descriptive tagline.
platforms/blabsy/src/components/view/view-tweet-stats.tsx (2)
59-59
: LGTM: Stats label updated for Blabsy branding.The change from "Retweet" to "Reblab" in the allStats array maintains consistency with the platform rebranding effort.
13-61
: LGTM: Code formatting improvements.The formatting and indentation changes improve code readability while preserving all component functionality and logic.
Also applies to: 63-119
platforms/blabsy/src/components/aside/aside.tsx (1)
13-13
: Approve early-return for small screens.
Theif (width < 1024) return null;
correctly hides the aside on narrower viewports.platforms/blabsy/src/components/layout/user-data-layout.tsx (1)
27-27
: Rebranded SEO title for missing user.
The<SEO>
title is correctly updated to'User not found / Blabsy'
to reflect the platform rename.platforms/blabsy/src/components/modal/tweet-stats-modal.tsx (1)
24-24
: Approve rebranding of retweet label.
The header title now uses'Reblabed'
instead of'Retweeted'
, aligning with the new terminology.platforms/blabsy/src/components/tweet/tweet-date.tsx (2)
2-2
: Verifycn
import source.
This file importscn
from'clsx'
rather than the shared utility—please confirm this is intentional.
21-33
: Skip false-positive a11y lint.
Next.jsLink
passes thehref
to its<a>
child automatically; the missinghref
warning is a false positive in this context.🧰 Tools
🪛 Biome (1.9.4)
[error] 22-28: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.(lint/a11y/useValidAnchor)
platforms/blabsy/src/pages/404.tsx (1)
13-13
: Rebranded 404 SEO title.
The<SEO>
title is updated to'Page not found / Blabsy'
, consistent with the renaming effort.platforms/blabsy/src/components/common/app-head.tsx (2)
6-7
: LGTM! Consistent rebranding of page metadata.The title and Open Graph title updates correctly reflect the platform rebranding from "Twitter" to "Blabsy".
10-11
: Note: Twitter metadata preserved for social cards.The
twitter:site
andtwitter:card
metadata remain unchanged, which appears intentional to maintain Twitter card functionality for social sharing. This is appropriate as these are Twitter-specific technical identifiers rather than user-facing text.platforms/blabsy/src/pages/home.tsx (2)
29-29
: LGTM! SEO title correctly updated for rebranding.The page title change from "Home / Twitter" to "Home / Blabsy" aligns perfectly with the platform rebranding effort.
19-55
: Formatting improvements enhance readability.The consistent 4-space indentation standardization improves code maintainability without affecting functionality.
platforms/blabsy/src/components/user/user-follow.tsx (2)
32-35
: LGTM! SEO title suffix correctly updated for rebranding.The title suffix change from "Twitter" to "Blabsy" maintains the dynamic user information while reflecting the new platform branding.
10-10
: Formatting consistency improved.The indentation adjustment for the
type
property aligns with the 4-space standard used throughout the codebase.platforms/blabsy/src/pages/user/[id]/likes.tsx (3)
33-35
: LGTM! SEO title comprehensively updated for rebranding.The title correctly changes both "Tweets liked by" to "Blabs liked by" and the platform suffix from "Twitter" to "Blabsy", maintaining the dynamic user information structure.
41-42
: LGTM! Empty state messaging updated consistently.The empty state messages correctly update terminology from "Tweets" to "Blabs" while preserving the helpful user guidance about when content will appear.
17-52
: Formatting standardization improves code consistency.The indentation updates to 4 spaces enhance readability and maintain consistency with the broader codebase standards.
platforms/blabsy/src/components/user/user-nav.tsx (1)
12-16
: LGTM! Consistent rebranding applied.The navigation labels have been properly updated from "Tweets" to "Blabs" and "Tweets & replies" to "Blabs & replies", maintaining consistency with the platform rebranding effort.
platforms/blabsy/src/components/modal/mobile-sidebar-modal.tsx (1)
21-36
: Intentional navigation link removal acknowledged.The commenting out of "Topics" and "Lists" navigation links aligns with the PR objective to remove non-functional component flows.
platforms/blabsy/src/pages/trends.tsx (2)
21-21
: LGTM! SEO title properly updated.The SEO title has been correctly updated from "Trends / Twitter" to "Trends / Blabsy" as part of the rebranding effort.
31-31
: Intentional component removal acknowledged.Commenting out the
AsideTrends
component aligns with the PR objective to remove non-functional component flows.platforms/blabsy/src/components/modal/display-modal.tsx (2)
32-35
: LGTM! Comprehensive branding update.The descriptive text has been properly updated to reference "Blabsy accounts" instead of Twitter, maintaining consistency with the platform rebranding.
55-60
: LGTM! Platform messaging consistently updated.The core platform description has been thoroughly updated from "Twitter/Tweets" to "Blabsy/Blabs" with proper mention handling (@blabsy), providing users with accurate information about the rebranded platform.
platforms/blabsy/src/components/aside/search-bar.tsx (1)
56-61
: Branding consistency: updated placeholder text
The search input placeholder has been correctly changed from "Search Twitter" to "Search Blabsy".platforms/blabsy/src/components/login/login-footer.tsx (1)
38-38
: Brand update: copyright notice
Updated to "© 2025 Blabsy" to reflect the new branding and current year.platforms/blabsy/src/pages/tweet/[id].tsx (2)
48-52
: Brand update: pageTitle uses Blabsy
The dynamicpageTitle
now correctly references "Blabsy" instead of "Twitter".
66-68
: Brand update: error messages
The error state now references "Blab not found" and "Blabsy" in the SEO title, which aligns correctly with the rebranding.platforms/blabsy/src/components/ui/follow-button.tsx (1)
42-45
: Brand update: updated modal description to "Blabs"
The unfollow confirmation description now correctly mentions "Blabs" instead of "Tweets".platforms/blabsy/src/components/aside/aside-footer.tsx (2)
1-10
: Consider whether footer navigation should be completely removed.All footer navigation links have been commented out, removing important legal and accessibility links like Terms of Service, Privacy Policy, etc. While the old Twitter links are no longer relevant, consider whether Blabsy-specific equivalents should be provided to maintain proper footer navigation.
Also applies to: 18-30
31-31
: Copyright update looks good.The copyright update from "© 2022 Twitter, Inc." to "© 2025 Blabsy" correctly reflects the rebranding and current year.
platforms/blabsy/src/components/sidebar/sidebar.tsx (2)
28-34
: Verify if navigation features should be permanently removed.The "Explore" and "Lists" navigation links have been commented out. Ensure this is intentional for the Blabsy platform and not a temporary removal during development.
Also applies to: 52-58
125-125
: Button text update aligns with rebranding.The button text change from "Tweet" to "Blab" correctly reflects the platform rebranding.
platforms/blabsy/src/components/input/input-form.tsx (2)
121-121
: Modal text update aligns with rebranding.The modal title change from "Discard Tweet?" to "Discard Blab?" correctly reflects the platform rebranding.
153-153
: Placeholder text update aligns with rebranding.The placeholder text change from "Tweet your reply" to "Blab your reply" correctly reflects the platform rebranding.
platforms/blabsy/src/components/input/input-options.tsx (1)
149-149
: Button label update aligns with rebranding.The button label change from "Tweet" to "Blab" (when not in reply mode) correctly reflects the platform rebranding while preserving the "Reply" label for reply contexts.
platforms/blabsy/functions/src/normalize-stats.ts (1)
48-48
: LGTM: Consistent rebranding terminologyThe log message correctly updates from "tweet" to "blab" aligning with the platform rebranding.
platforms/blabsy/src/components/user/user-cards.tsx (1)
24-46
: LGTM: Comprehensive and consistent rebrandingThe updates to
allNoStatsData
consistently replace "Tweets/Retweets" with "Blabs/Reblabs" across titles, descriptions, and image assets. The rebranding is thorough and maintains proper context in all user-facing messages.platforms/blabsy/src/pages/bookmarks.tsx (2)
60-60
: LGTM: SEO title properly updated for rebrandingThe page title correctly changes from "Twitter" to "Blabsy" maintaining consistency with the platform rebranding.
104-105
: LGTM: Empty state text consistently updatedThe StatsEmpty component properly uses "Blabs" terminology instead of "Tweets" in both title and description.
platforms/blabsy/src/pages/user/[id]/with_replies.tsx (2)
44-46
: LGTM: SEO metadata properly updated for rebrandingThe SEO title correctly updates from "Tweets with replies" to "Blabs with replies" and from "Twitter" to "Blabsy", maintaining consistency with the platform rebranding.
52-53
: LGTM: Empty state messaging consistently updatedThe StatsEmpty component properly updates from "tweeted" to "blabbed" and "Tweets" to "Blabs", maintaining natural language flow while implementing the rebranding terminology.
platforms/blabsy/src/components/tweet/tweet-stats.tsx (1)
115-115
: LGTM! Consistent rebranding from "Retweet" to "Reblab".The tooltip text update aligns perfectly with the platform-wide rebranding effort.
platforms/blabsy/src/pages/people.tsx (2)
35-35
: LGTM! SEO title updated for consistent branding.The title change from "People / Twitter" to "People / Blabsy" aligns with the platform-wide rebranding effort.
8-10
: LGTM! Improved code formatting and consistency.The indentation and formatting improvements enhance code readability while maintaining all functionality.
Also applies to: 22-32, 33-58, 62-67
platforms/blabsy/README.md (2)
17-24
: LGTM! Comprehensive rebranding from "tweets" to "blabs".The terminology updates are consistent and thorough, properly reflecting the platform's new branding.
🧰 Tools
🪛 LanguageTool
[duplication] ~20-~20: Possible typo: you repeated a word.
Context: ...- Users can follow and unfollow other users - Users can see their and other followers and t...(ENGLISH_WORD_REPEAT_RULE)
47-47
: LGTM! Repository URL updated correctly.The clone URL now points to the correct MetaState-Prototype-Project repository.
platforms/blabsy/src/components/layout/common-layout.tsx (2)
9-9
: LGTM! Improved formatting for type definition.The indentation improvement enhances code readability.
25-25
: Coordinated removal of AsideTrends feature.The commenting out of AsideTrends components appears to be part of a coordinated effort to disable the trends feature across the platform, which aligns with similar changes in the aside.tsx component.
Also applies to: 38-38
platforms/blabsy/src/components/tweet/tweet-share.tsx (1)
129-129
: LGTM! Consistent rebranding terminology.The change from "Remove Tweet from Bookmarks" to "Remove Blab from Bookmarks" aligns perfectly with the platform rebranding effort.
platforms/blabsy/src/components/tweet/tweet-actions.tsx (2)
120-121
: LGTM! Consistent rebranding terminology.The success message correctly uses "blab" instead of "tweet" terminology.
133-136
: LGTM! Consistent rebranding terminology.The toast message correctly uses "blab" terminology for pin/unpin actions.
platforms/blabsy/src/components/tweet/tweet.tsx (4)
110-110
: LGTM! Consistent rebranding terminology.The change from "Pinned Tweet" to "Pinned Blab" maintains consistency with the platform rebranding.
121-121
: LGTM! Consistent rebranding terminology.The change from "Retweeted" to "Reblabed" aligns with the new platform terminology.
116-116
: Good type safety improvement.Adding the explicit string cast for
profileUsername
addresses potential undefined value issues.
36-36
:❌ Incorrect review comment
Consider renaming to avoid declaration conflict.
Static analysis flags a redeclaration issue with 'Tweet'. Consider renaming the function or using a namespace to avoid conflicts.
Run this script to check for other Tweet declarations:
🏁 Script executed:
#!/bin/bash # Check for other Tweet type or function declarations that might conflict ast-grep --pattern 'function Tweet($_) { $$$ }' ast-grep --pattern 'type Tweet = $$$' ast-grep --pattern 'interface Tweet { $$$ }'Length of output: 1758
🏁 Script executed:
#!/bin/bash # Display imports and function signature from tweet.tsx sed -n '1,50p' platforms/blabsy/src/components/tweet/tweet.tsx | sed -n '1,50p'Length of output: 1713
No redeclaration conflict between imported
Tweet
type andTweet
function
TypeScript separates the type and value namespaces. Theimport type { Tweet }
is erased at compile time (type-only), whileexport function Tweet(…)
lives in the value namespace. There’s no duplicate identifier error, so no renaming is needed.Likely an incorrect or invalid review comment.
🧰 Tools
🪛 Biome (1.9.4)
[error] 36-36: Shouldn't redeclare 'Tweet'. Consider to delete it or rename it.
'Tweet' is defined here:
(lint/suspicious/noRedeclare)
platforms/blabsy/src/components/input/input.tsx (1)
117-117
: LGTM! Consistent rebranding terminology.The success toast message correctly uses "Blab" instead of "Tweet" terminology.
platforms/blabsy/src/components/home/update-username.tsx (1)
133-133
: LGTM! Consistent rebranding terminology.The tooltip text correctly changes from "Top tweets" to "Top blabs" maintaining consistency with the platform rebranding.
}, | ||
}); | ||
|
||
const tweetLink = `https://twitter-clone-ccrsxx.vercel.app/tweet/${snapshot.id}`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
Verify URL domain consistency with Blabsy rebranding.
The email contains a link to twitter-clone-ccrsxx.vercel.app
which may be inconsistent with the Blabsy rebranding effort. Consider updating the domain to reflect the new branding if applicable.
🏁 Script executed:
#!/bin/bash
# Description: Check if this domain URL appears elsewhere in the codebase and verify if it should be updated
# Expected: Find other references to determine if domain should be updated for consistency
# Search for the specific domain
rg "twitter-clone-ccrsxx.vercel.app" --type ts --type tsx --type js --type jsx
# Search for any environment variables or config files that might define the base URL
rg "NEXT_PUBLIC_BASE_URL|BASE_URL|DOMAIN" --type env --type ts --type js
Length of output: 209
I’ll broaden the search to catch all occurrences and any config references:
🏁 Script executed:
#!/usr/bin/env bash
# 1. Search for the hard-coded domain anywhere in the repo
rg "twitter-clone-ccrsxx.vercel.app"
# 2. Look for environment/config variables that could define the base URL or domain
rg -i "BASE_URL|NEXT_PUBLIC_BASE_URL|DOMAIN" -g '*.env' -g '*.json' -g '*.ts' -g '*.js'
Length of output: 1682
Use a configurable Blabsy base URL instead of hard-coding the old domain
It looks like notify-email.ts
is still generating links with twitter-clone-ccrsxx.vercel.app
, and there isn’t a matching environment variable in this function. To keep branding consistent and make future updates painless:
-
Define (or reuse) an env var, e.g.
PUBLIC_BLABSY_BASE_URL
, for the Blabsy site URL. -
Update
platforms/blabsy/functions/src/notify-email.ts
to use that var with a sensible default:// platforms/blabsy/functions/src/notify-email.ts // before
-
const tweetLink = `https://twitter-clone-ccrsxx.vercel.app/tweet/${snapshot.id}`; // after
-
const baseUrl =
-
process.env.PUBLIC_BLABSY_BASE_URL ||
-
'https://twitter-clone-ccrsxx.vercel.app';
-
const tweetLink = `${baseUrl}/tweet/${snapshot.id}`;
This ensures all outbound links adopt the correct Blabsy domain and can be updated centrally.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
const tweetLink = `https://twitter-clone-ccrsxx.vercel.app/tweet/${snapshot.id}`; | |
// platforms/blabsy/functions/src/notify-email.ts | |
// replace hard-coded tweetLink with a configurable Blabsy base URL | |
const baseUrl = | |
process.env.PUBLIC_BLABSY_BASE_URL || | |
'https://twitter-clone-ccrsxx.vercel.app'; | |
const tweetLink = `${baseUrl}/tweet/${snapshot.id}`; |
🤖 Prompt for AI Agents
In platforms/blabsy/functions/src/notify-email.ts at line 27, the tweet link is
hard-coded with the old domain twitter-clone-ccrsxx.vercel.app, which is
inconsistent with the Blabsy rebranding. To fix this, introduce an environment
variable like PUBLIC_BLABSY_BASE_URL to hold the base URL for Blabsy, provide a
sensible default if the env var is not set, and update the tweetLink
construction to use this variable instead of the hard-coded domain. This change
centralizes domain configuration and ensures branding consistency.
blurClassName='bg-accent-blue' | ||
src='/assets/twitter-banner.png' | ||
alt='Twitter banner' | ||
alt='Blabsy banner' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💡 Verification agent
🧩 Analysis chain
LGTM: Alt text updated for Blabsy branding.
The alt attribute change from "Twitter banner" to "Blabsy banner" aligns with the rebranding effort.
However, note that the image source still references /assets/twitter-banner.png
. Consider renaming the image file to maintain consistency with the Blabsy branding.
🏁 Script executed:
#!/bin/bash
# Description: Check if the twitter-banner.png file exists and find other references
# Expected: Verify file existence and find other uses to assess renaming impact
# Check if the image file exists in common asset directories
fd "twitter-banner.png" --type f
# Search for references to this image across the codebase
rg "twitter-banner\.png" --type ts --type tsx --type js --type jsx
Length of output: 179
Address image filename consistency with Blabsy branding
The new alt text is correct, but the source file still uses the old “twitter-banner.png” name. To complete the rebrand:
- Rename
platforms/blabsy/public/assets/twitter-banner.png
→platforms/blabsy/public/assets/blabsy-banner.png
- Update the reference in
platforms/blabsy/src/components/login/login-main.tsx
- <img src="/assets/twitter-banner.png" alt="Blabsy banner" /> + <img src="/assets/blabsy-banner.png" alt="Blabsy banner" />
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
alt='Blabsy banner' | |
<img src="/assets/blabsy-banner.png" alt="Blabsy banner" /> |
🤖 Prompt for AI Agents
In platforms/blabsy/src/components/login/login-main.tsx at line 54, the image
alt text was updated to "Blabsy banner" but the image source still points to
"twitter-banner.png". To fix this, rename the image file located at
platforms/blabsy/public/assets/twitter-banner.png to blabsy-banner.png and
update the image source path in login-main.tsx accordingly to maintain branding
consistency.
<button | ||
className={cn( | ||
`hover-animation mt-0.5 mb-[3px] flex h-4 items-center gap-1 border-b | ||
border-b-transparent outline-none hover:border-b-light-primary | ||
focus-visible:border-b-light-primary dark:hover:border-b-dark-primary | ||
dark:focus-visible:border-b-dark-primary`, | ||
index === 0 && 'cursor-not-allowed' | ||
)} | ||
key={title} | ||
onClick={type ? handleOpen(type) : undefined} | ||
> | ||
<NumberStats move={move} stats={stats} /> | ||
<p>{`${ | ||
stats === 1 | ||
? title | ||
: stats > 1 && index === 0 | ||
? `${title.slice(0, -1)}ies` | ||
: `${title}s` | ||
}`}</p> | ||
</button> | ||
) | ||
)} | ||
</div> | ||
)} | ||
</> | ||
); | ||
index === 0 && 'cursor-not-allowed' | ||
)} | ||
key={title} | ||
onClick={ | ||
type ? handleOpen(type) : undefined | ||
} | ||
> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Add explicit button type to prevent form submission.
The button element should have an explicit type="button"
to prevent unintended form submissions when placed inside a form element.
<button
+ type="button"
className={cn(
`hover-animation mt-0.5 mb-[3px] flex h-4 items-center gap-1 border-b
border-b-transparent outline-none hover:border-b-light-primary
focus-visible:border-b-light-primary dark:hover:border-b-dark-primary
dark:focus-visible:border-b-dark-primary`,
index === 0 && 'cursor-not-allowed'
)}
key={title}
onClick={
type ? handleOpen(type) : undefined
}
>
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
<button | |
className={cn( | |
`hover-animation mt-0.5 mb-[3px] flex h-4 items-center gap-1 border-b | |
border-b-transparent outline-none hover:border-b-light-primary | |
focus-visible:border-b-light-primary dark:hover:border-b-dark-primary | |
dark:focus-visible:border-b-dark-primary`, | |
index === 0 && 'cursor-not-allowed' | |
)} | |
key={title} | |
onClick={type ? handleOpen(type) : undefined} | |
> | |
<NumberStats move={move} stats={stats} /> | |
<p>{`${ | |
stats === 1 | |
? title | |
: stats > 1 && index === 0 | |
? `${title.slice(0, -1)}ies` | |
: `${title}s` | |
}`}</p> | |
</button> | |
) | |
)} | |
</div> | |
)} | |
</> | |
); | |
index === 0 && 'cursor-not-allowed' | |
)} | |
key={title} | |
onClick={ | |
type ? handleOpen(type) : undefined | |
} | |
> | |
<button | |
type="button" | |
className={cn( | |
`hover-animation mt-0.5 mb-[3px] flex h-4 items-center gap-1 border-b | |
border-b-transparent outline-none hover:border-b-light-primary | |
focus-visible:border-b-light-primary dark:hover:border-b-dark-primary | |
dark:focus-visible:border-b-dark-primary`, | |
index === 0 && 'cursor-not-allowed' | |
)} | |
key={title} | |
onClick={ | |
type ? handleOpen(type) : undefined | |
} | |
> |
🧰 Tools
🪛 Biome (1.9.4)
[error] 92-104: Provide an explicit type prop for the button element.
The default type of a button is submit, which causes the submission of a form when placed inside a form
element. This is likely not the behaviour that you want inside a React application.
Allowed button types are: submit, button or reset
(lint/a11y/useButtonType)
🤖 Prompt for AI Agents
In platforms/blabsy/src/components/view/view-tweet-stats.tsx around lines 92 to
104, the button element lacks an explicit type attribute, which can cause it to
submit a form unintentionally if inside a form. Add type="button" to the button
element to prevent it from acting as a submit button by default.
title='Log out of Twitter?' | ||
description='You can always log back in at any time. If you just want to switch accounts, you can do that by adding an existing account.' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Update modal text to reflect Blabsy branding.
The action modal still references "Twitter" in the title and description, which should be updated to "Blabsy" for consistency.
- title='Log out of Twitter?'
- description='You can always log back in at any time. If you just want to switch accounts, you can do that by adding an existing account.'
+ title='Log out of Blabsy?'
+ description='You can always log back in at any time. If you just want to switch accounts, you can do that by adding an existing account.'
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
title='Log out of Twitter?' | |
description='You can always log back in at any time. If you just want to switch accounts, you can do that by adding an existing account.' | |
title='Log out of Blabsy?' | |
description='You can always log back in at any time. If you just want to switch accounts, you can do that by adding an existing account.' |
🤖 Prompt for AI Agents
In platforms/blabsy/src/components/modal/mobile-sidebar-modal.tsx around lines
123 to 124, update the modal title and description text to replace all instances
of "Twitter" with "Blabsy" to ensure the branding is consistent throughout the
UI.
// }, | ||
{ | ||
href: '/people', | ||
linkName: 'Twitter Circle', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Update "Twitter Circle" to maintain branding consistency.
The link name still references "Twitter Circle" which should be updated to "Blabsy Circle" to align with the rebranding effort.
- linkName: 'Twitter Circle',
+ linkName: 'Blabsy Circle',
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
linkName: 'Twitter Circle', | |
linkName: 'Blabsy Circle', |
🤖 Prompt for AI Agents
In platforms/blabsy/src/components/modal/mobile-sidebar-modal.tsx at line 39,
update the linkName value from "Twitter Circle" to "Blabsy Circle" to maintain
branding consistency with the rebranding effort.
tweetIsRetweeted && ( | ||
<TweetStatus type='tweet'> | ||
<Link href={profileUsername as string}> | ||
<a className='custom-underline truncate text-sm font-bold'> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix missing href attribute for accessibility.
The static analysis correctly identifies missing href
attribute on the anchor element.
Apply this fix:
- <a className='custom-underline truncate text-sm font-bold'>
+ <a href={`/user/${profileUsername as string}`} className='custom-underline truncate text-sm font-bold'>
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
<a className='custom-underline truncate text-sm font-bold'> | |
<a href={`/user/${profileUsername as string}`} className='custom-underline truncate text-sm font-bold'> |
🧰 Tools
🪛 Biome (1.9.4)
[error] 117-117: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
🤖 Prompt for AI Agents
In platforms/blabsy/src/components/tweet/tweet.tsx at line 117, the anchor
element is missing an href attribute, which is important for accessibility and
proper HTML semantics. Add a valid href attribute to the <a> tag, such as
href="#" or the appropriate URL, to ensure the link is accessible and behaves
correctly.
<span className='flex gap-2'> | ||
Your Blab was sent | ||
<Link href={`/tweet/${tweetId}`}> | ||
<a className='custom-underline font-bold'>View</a> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix missing href attribute for accessibility.
The anchor element needs an href
attribute for proper accessibility.
Apply this fix:
- <a className='custom-underline font-bold'>View</a>
+ <a href={`/tweet/${tweetId}`} className='custom-underline font-bold'>View</a>
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
<a className='custom-underline font-bold'>View</a> | |
<a href={`/tweet/${tweetId}`} className='custom-underline font-bold'>View</a> |
🧰 Tools
🪛 Biome (1.9.4)
[error] 119-119: Provide a href attribute for the a element.
An anchor element should always have a href
Check this thorough explanation to better understand the context.
(lint/a11y/useValidAnchor)
🤖 Prompt for AI Agents
In platforms/blabsy/src/components/input/input.tsx at line 119, the anchor tag
is missing an href attribute, which is necessary for accessibility. Add a valid
href attribute to the <a> element, such as href="#" or the appropriate URL, to
ensure it is accessible and behaves correctly as a link.
useEffect( | ||
() => { | ||
if (modal) inputRef.current?.focus(); | ||
return cleanImage; | ||
}, | ||
// eslint-disable-next-line react-hooks/exhaustive-deps | ||
[] | ||
); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Fix hook dependency to prevent stale closures.
The useEffect
hook is missing the modal
dependency which could lead to stale closure issues.
Apply this fix:
useEffect(
() => {
if (modal) inputRef.current?.focus();
return cleanImage;
},
- // eslint-disable-next-line react-hooks/exhaustive-deps
- []
+ [modal]
);
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
useEffect( | |
() => { | |
if (modal) inputRef.current?.focus(); | |
return cleanImage; | |
}, | |
// eslint-disable-next-line react-hooks/exhaustive-deps | |
[] | |
); | |
useEffect( | |
() => { | |
if (modal) inputRef.current?.focus(); | |
return cleanImage; | |
}, | |
[modal] | |
); |
🧰 Tools
🪛 Biome (1.9.4)
[error] 66-66: This hook does not specify all of its dependencies: modal
This dependency is not specified in the hook dependency list.
Either include it or remove the dependency array
(lint/correctness/useExhaustiveDependencies)
🤖 Prompt for AI Agents
In platforms/blabsy/src/components/input/input.tsx around lines 66 to 73, the
useEffect hook is missing the modal dependency, which can cause stale closure
issues. Add modal to the dependency array of the useEffect hook to ensure it
runs whenever modal changes, preventing stale references.
useEffect(() => { | ||
if (!user?.updatedAt) openModal(); | ||
else setAlreadySet(true); | ||
}, []); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Fix hook dependencies for proper effect behavior.
The useEffect
hook is missing dependencies that could cause it to not run when expected.
Apply this fix:
useEffect(() => {
if (!user?.updatedAt) openModal();
else setAlreadySet(true);
- }, []);
+ }, [user?.updatedAt, openModal]);
🧰 Tools
🪛 Biome (1.9.4)
[error] 61-61: This hook does not specify all of its dependencies: openModal
This dependency is not specified in the hook dependency list.
Either include it or remove the dependency array
(lint/correctness/useExhaustiveDependencies)
[error] 61-61: This hook does not specify all of its dependencies: user?.updatedAt
This dependency is not specified in the hook dependency list.
Either include it or remove the dependency array
(lint/correctness/useExhaustiveDependencies)
🤖 Prompt for AI Agents
In platforms/blabsy/src/components/home/update-username.tsx around lines 61 to
64, the useEffect hook is missing dependencies, which can cause it to not run
correctly when the user data changes. Add the necessary dependencies,
specifically the user object or user.updatedAt, to the dependency array so the
effect runs whenever these values change.
useEffect(() => { | ||
const checkAvailability = async (value: string): Promise<void> => { | ||
setSearching(true); | ||
|
||
const empty = await checkUsernameAvailability(value); | ||
const empty = await checkUsernameAvailability(value); | ||
|
||
if (empty) setAvailable(true); | ||
else { | ||
setAvailable(false); | ||
setErrorMessage('This username has been taken. Please choose another.'); | ||
} | ||
if (empty) setAvailable(true); | ||
else { | ||
setAvailable(false); | ||
setErrorMessage( | ||
'This username has been taken. Please choose another.' | ||
); | ||
} | ||
|
||
setSearching(false); | ||
}; | ||
setSearching(false); | ||
}; | ||
|
||
if (!visited && inputValue.length > 0) setVisited(true); | ||
|
||
if (visited) { | ||
if (errorMessage) setErrorMessage(''); | ||
|
||
const error = isValidUsername(user?.username as string, inputValue); | ||
|
||
if (error) { | ||
setAvailable(false); | ||
setErrorMessage(error); | ||
} else void checkAvailability(inputValue); | ||
} | ||
}, [inputValue]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix hook dependencies to prevent stale closures.
The useEffect
hook is missing several dependencies that could lead to stale closure issues and incorrect behavior.
Apply this fix:
useEffect(() => {
const checkAvailability = async (value: string): Promise<void> => {
setSearching(true);
const empty = await checkUsernameAvailability(value);
if (empty) setAvailable(true);
else {
setAvailable(false);
setErrorMessage(
'This username has been taken. Please choose another.'
);
}
setSearching(false);
};
if (!visited && inputValue.length > 0) setVisited(true);
if (visited) {
if (errorMessage) setErrorMessage('');
const error = isValidUsername(user?.username as string, inputValue);
if (error) {
setAvailable(false);
setErrorMessage(error);
} else void checkAvailability(inputValue);
}
- }, [inputValue]);
+ }, [inputValue, visited, errorMessage, user?.username]);
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
useEffect(() => { | |
const checkAvailability = async (value: string): Promise<void> => { | |
setSearching(true); | |
const empty = await checkUsernameAvailability(value); | |
const empty = await checkUsernameAvailability(value); | |
if (empty) setAvailable(true); | |
else { | |
setAvailable(false); | |
setErrorMessage('This username has been taken. Please choose another.'); | |
} | |
if (empty) setAvailable(true); | |
else { | |
setAvailable(false); | |
setErrorMessage( | |
'This username has been taken. Please choose another.' | |
); | |
} | |
setSearching(false); | |
}; | |
setSearching(false); | |
}; | |
if (!visited && inputValue.length > 0) setVisited(true); | |
if (visited) { | |
if (errorMessage) setErrorMessage(''); | |
const error = isValidUsername(user?.username as string, inputValue); | |
if (error) { | |
setAvailable(false); | |
setErrorMessage(error); | |
} else void checkAvailability(inputValue); | |
} | |
}, [inputValue]); | |
useEffect(() => { | |
const checkAvailability = async (value: string): Promise<void> => { | |
setSearching(true); | |
const empty = await checkUsernameAvailability(value); | |
if (empty) setAvailable(true); | |
else { | |
setAvailable(false); | |
setErrorMessage( | |
'This username has been taken. Please choose another.' | |
); | |
} | |
setSearching(false); | |
}; | |
if (!visited && inputValue.length > 0) setVisited(true); | |
if (visited) { | |
if (errorMessage) setErrorMessage(''); | |
const error = isValidUsername(user?.username as string, inputValue); | |
if (error) { | |
setAvailable(false); | |
setErrorMessage(error); | |
} else void checkAvailability(inputValue); | |
} | |
}, [inputValue, visited, errorMessage, user?.username]); |
🧰 Tools
🪛 Biome (1.9.4)
[error] 30-30: This hook does not specify all of its dependencies: errorMessage
This dependency is not specified in the hook dependency list.
(lint/correctness/useExhaustiveDependencies)
[error] 30-30: This hook does not specify all of its dependencies: user?.username
This dependency is not specified in the hook dependency list.
(lint/correctness/useExhaustiveDependencies)
[error] 30-30: This hook does not specify all of its dependencies: visited
This dependency is not specified in the hook dependency list.
This dependency is not specified in the hook dependency list.
(lint/correctness/useExhaustiveDependencies)
🤖 Prompt for AI Agents
In platforms/blabsy/src/components/home/update-username.tsx around lines 30 to
59, the useEffect hook is missing dependencies such as visited, errorMessage,
user, and the functions checkUsernameAvailability and isValidUsername, which can
cause stale closures and incorrect behavior. Update the dependency array to
include all these variables and functions used inside the effect to ensure it
runs correctly when any of them change.
* initial commit * chore: add w3id readme (#3) * chore: add w3id readme * chore: bold text * chore: better formatting * docs: add w3id details * chore: format * chore: add links * fix: id spec considerations addressal (#8) * fix: id spec considerations addressal * fix: identity -> indentifier * chore: expand on trust list based recovery * chore: expand on AKA --------- Co-authored-by: Merul Dhiman <[email protected]> * Docs/eid wallet (#10) * chore: add eid-wallet folder * chore: add eid wallet docs * feat: add (#9) * feat(w3id): basic setup (#11) * feat(w3id): basic setup * fix(root): add infrastructure workspaces * update: lock file * feat(eidw): setup tauri (#40) * Feat/setup daisyui (#46) * feat: setup-daisyui * fix: index file * feat: colors added * feat: Archivo font added * fix: postcss added * fix: +layout.svelte file added * fix: packages * fix: fully migrating to tailwind v4 * feat: add Archivo font * feat: add danger colors * feat: twmerge and clsx added * feat: shadcn function added --------- Co-authored-by: Bekiboo <[email protected]> Co-authored-by: Julien <[email protected]> * feat: add storybook (#45) * feat: add storybook * update: lockfile * feat: created connection button (#48) * created connection button * added restprops to parent class * added onClick btn and storybook * fix: make font work in storybook (#54) * Feat/header (#55) * feat: add icons lib * fix: make font work in storybook * feat: Header * feat: runtime global added, icon library created, icons added, type file added * feat: header props added * fix: remove icons and type file as we are using lib for icons * fix: heading style * fix: color and icons, git merge branch 51, 54 * fix: color * fix: header-styling * fix: classes * chore: handlers added * chore: handlers added * fix: added heading --------- Co-authored-by: Soham Jaiswal <[email protected]> * Alternative w3id diagram (#52) * Feat/cupertino pane (#49) * feat: Drawer * feat: Drawer and added a function for clickoutside in utils * fix: classes * fix: drawer button position * fix: style and clickoutside * fix: pane height * fix: border-radius * fix: drawer as bulletin * fix: styling * fix: component with inbuilt features * fix: remove redundant code * fix: remove redundant code * fix: cancel button * fix: css in storybook * fix: position * fix: height of pane * fix: remove redundant code * feat: add button action component (#47) * feat: add button action component * fix: add correct weights to Archivo fontt * feat: add base button * fix: set prop classes last * feat: improve loading state * chore: cleanup * feat: add button action component * fix: add correct weights to Archivo fontt * feat: add base button * fix: set prop classes last * feat: improve loading state * chore: cleanup * chore: add documentation * fix: configure Storybook * chore: storybook gunk removal * feat: enhance ButtonAction component with type prop and better error handling --------- Co-authored-by: JulienAuvo <[email protected]> * Feat/splash screen (#63) * feat: SplashScreen * fix: remove redundant code * fix: as per given suggestion * fix: font-size * fix: logo * feat: input-pin (#56) * feat: input-pin * fix: styling as per our design * fix: added small variant * fix: hide pin on select * fix: gap between pins * fix: color of focus state * fix: removed legacy code and also fix some css to tailwind css * fix: css * fix: optional props * feat: added color variants * Feat/improve button component (#60) * feat: add white variant * feat: add small variant * chore: update doc and story for button * chore: rename cb into callback * update: improve small size * update: modify loading style * fix: return getAbsolutePath function to storybook (#58) Co-authored-by: Bekiboo <[email protected]> * feat: add selector component (#59) * feat: add selector component * feat: improve selector + add flag-icon lib * feat: improve selector + doc * feat: add utility function to get language with country name * feat: test page for language selectors * chore: add Selector Story * chore: clean test page * fix: types * fix: normalize custom tailwind colors (#71) * feat: workflows (#64) * feat: workflows * fix: node version * fix: use pnpm 10 * fix: check message * Fix/codebase linting (#73) * fix: Check Lint / lint * fix: Check Lint / lint * fix: Check Lint / lint * fix: Check Lint / lint * fix: Check Code / lint * fix: Check Format / lint * fix: Check Code / lint * fix: Check Format / lint * fix: Check Code / lint * fix: Check Format / lint * fix: Check Code / lint * fix: Check Code / lint * fix: Check Format / lint * fix: unknown property warning * fix: unknown property warning * chore: improve args type * settings nav button :) (#75) * setting bav button all done :) * lint fixski * added component to index.ts * Feat/#32 identity card fragment (#74) * identity card * identity card * lint fixski * lint fixski * lint fixski * fixed the font weight * added component to index.ts * changed span to buttton * feat: add icon button component (#68) * feat: add icon button component * feat: finish up buttonIcon + stories * fix: update with new color naming * feat: polish button icon (and button action too) * chore: format lint * chore: sort imports * chore: format, not sure why * Feat/onboarding flow (#67) * feat: onboarding-page * fix: line height and added handlers * fix: button variant * fix: text-decoration * fix: subtext * fix: underline * fix: padding and button spacing * fix: according to design update * feat: Drawer * feat: verify-pae * fix: verify-page styling * feat: drawer for both confirm pin and add bio metrics added * feat: modal added in fragments * fix: icons and flow * feat: Identifier Card * fix: copy to clipboard * feat: e-passport page * fix: error state * fix: colors * fix: lint error * fix: lint * feat: Typography * fix: typograpy * fix: as per given suggestion * fix: font-sizing * fix: identity card implementation * fix: spacing * fix: padding * fix: padding and spacing * fix: splashscreen * fix: error state * fix: styling to avoid * fix:typo * Fix/remove daisyui (#82) * refactoring: remove DaisyUI + refactor some tailwind classes and logic * refactoring: remove DaisyUI + refactor some tailwind classes and logic * feat: add Button.Nav (#77) * feat: add Button.Nav * chore: format * chore: sort imports * update: remove unused snippet and add missing props * feat: stick to fragment definition * update: documentation * fix: stories * chore: sort imports * Feat/splashscreen animation (#81) * feat: add animation to splashScreen * feat: implement data loading logic with splash screen delay * chore: sort import * update: use ButtonIcon is IdentityCard (#78) * update: use ButtonIcon is IdentityCard * feat: refactor ButtonIcon to be used anywhere in the app * chore: format indent * chore: remove useless change * feat: setup safe area (#80) * feat: setup safe area * chore: simplify implementation * chore: format * Feat/uuidv5 generation (#61) * feat: setup uuidv5 * chore: add test for deterministic UUID * feat: add Hero fragment (#88) * feat: add Hero fragment * chore: sort imports + add doc * feat: add storage specification abstract class (#92) * feat: add storage specification abstract class * chore: format and ignore lint * chore: change format checker on w3id * feat: settings-flow (#86) * feat: settings-flow * feat: settings and language page * feat : history page * feat: change pin page * fix: height of selector * fix: pin change page * fix: size of input pin * fix: spacing of pins * feat: AppNav fragment * fix: height of page * fix: padding * fix: remove redundant code * feat: privacy page * chore: add doc * fix: error state * feat: remove redundant code * chore: used app nav component --------- Co-authored-by: JulienAuvo <[email protected]> * feat: AppNav fragment (#90) * feat: AppNav fragment * chore: add doc * feat: Main page flow (#93) * feat: create root page + layout * feat: complete main page flow beta * chore: fix ts block * chore: sort imports * feat: integrate-flows (#94) * feat: intigrate-flows * fix: spacing in e-passport page * fix: page connectivity * feat: app page transitions * fix: z index * fix: pages * fix: view transition effect on splashscreen * fix: drawer pill and cancel button removed * fix: share button removed when onboarding * fix: remove share and view button when on onboarding flow * fix: remove view button * fix: ci checks * fix: transitions * fix: transititon according to direction * fix: lint error * fix: loop holes * Feat/w3id log generation (#98) * chore: create basic log generation mechanism * chore: add hashing utility function * chore: rotation event * feat: genesis entry * feat: generalize hash function * feat: append entry * chore: basic tests * chore: add tests for rotation * feat: add malform throws * chore: add the right errors * chore: fix CI stuff * chore: add missing file * chore: fix event type enum * chore: format * feat: add proper error * chore: format * chore: remove eventtypes enum * chore: add new error for bad options * chore: add options tests * feat: add codec tests * fix: err handling && jsdoc * fix: run format * fix: remove unused import * fix: improve default error messages * fix: move redundant logic to function * fix: run format * fix: type shadow * fix: useless conversion/cast * fix: run format --------- Co-authored-by: Soham Jaiswal <[email protected]> * Feat/core id creation logic (#99) * feat: create w3id builder * fix: w3id builder * feat: add global config var for w3id * chore: add docs * chore: change rand to crng * chore: add ts type again * chore: fix lint and format * chore: add w3id tests github workflow * Feat/evault core (#100) * feat: migrate neo4j * chore: envelope logic works * chore: envelope logic works * feat: parsed envelopes search * feat: generics * feat: protocol * feat: jwt sigs in w3id * chore: stuff works * chore: tests for evault core * chore: format * chore: fix test * Feat/docker compose and docs (#101) * chore: stash dockerfile progress * fix: getEnvelopesByOntology thing * chore: fix tests * Update infrastructure/evault-core/src/protocol/vault-access-guard.ts Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * chore: remove unused import * chore: remove package * chore: fix pnpm lock * chore: fix workflow * chore: fix port in dockerfile --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Feat/registry and evault provisioning (#106) * feat: evault provisioning * chore: fianlly fixed provisioner * feat: add logic for metadata in consul * feat: registry * chore: format * Feat/watchers logs (#114) * feat: alloc according to entropy and namespace * chore: move exports * chore: docs * feat: `whois` endpoint * feat: watcher endpoints * chore: fix format and lint * chore: fix tests * feat: web3 adapter (#115) * feat: tauri plugins setup (#97) * feat: tauri plugins setup * fix: add editorconfig * fix: add missing biome json * fix: run formatter * feat: biometry homework * feat: add pin set logic * feat: add biometric enabling logic * fix: sec controller qol * feat: stub user controller * fix: run format && lint * fix: sort imports * fix: import statement sort * feat: user controller * feat: pin flow * feat: biometrics unavailable * fix: pin input not working * feat: make checks pass * fix: scan works * fix: actions * feat: format on save * fix: coderabbit suggestions * chore: run format lint check * fix: scan on decline too * feat: documentation links (#117) * feat: bad namespace test (#116) * fix: layouts (#119) * fix: layouts * fix: Onboarding page scroll fixed * fix: page layout and prevent from scroll in all devices * fix: pages layout * chore: try to fix emulator * fix: units * fix: safezones for ios * fix: styling --------- Co-authored-by: Soham Jaiswal <[email protected]> * feat: setup-metagram (#121) * feat: setup-metagram * chore: tailwind css worked * feat: fonts added * feat: typography * fix: removed stories and fixed setup for icons lib * feat: icons and story file * fix: type of args in story * fix: lint errors * feat: colors added * feat: Button * fix: format and lint * fix: colors * fix: spinner * fix: code rebbit suggestions * fix: code rebbit suggestions * fix: paraglide removed * fix: lock file * feat: added user avatar. (#130) * feat: Button (#129) * feat: Button * fix: colors of variants * feat: Input (#131) * feat: Input * feat: styling added * fix: styling * fix: styling * fix: added a new story * fix: focus states * fix: input states * Feat/settings navigation button (#140) * feat: settings-navigation-button * fix: handler added * chore: another variant added * fix: as per given suggestion * feat: BottomNav (#132) * feat: BottomNav * fix: icons * feat: profile icons created * feat: handler added * feat: handler added * fix: correct tags * fix: as per given suggestion, bottomnav moved to fragments and also implemented on page * fix: handler * chore: routes added * feat: app transitions added * fix: direction of transition * fix: transition css * fix: directionable transition * fix: used button instead of label, and used page from state * feat: added post fragment. (#137) * feat: FileInput (#150) * feat: FileInput * fix: added icon * feat: cancel upload * fix: remove redundant code * fix: usage docs added and as per requirements ' * fix: moved to framents * feat: Toggle Switch (#143) * feat: Toggle Switch * feat: Toggle Switch * fix: as per our design * fix: as per our design * feat: Label (#146) * feat: Select (#148) * feat: Select * fix: as per our design * fix: code format and as per svelte 5 * fix: font-size * fix: font-size * fix: icon * feat: message-input (#144) * feat: message-input * fix: classes merge and a files as a prop * feat: variant added * feat: icon replaced * fix: as per code rabbit suggestions * fix: icon * fix: input file button * fix: as per suggestion * fix: classes * fix: no need of error and disabled classes * fix: input * feat: invalid inputs * feat: add number input storybook --------- Co-authored-by: Soham Jaiswal <[email protected]> * feat:Drawer (#152) * feat:Drawer * feat: Drawer with clickoutside * fix: settings * Feat/metagram header (#133) * feat: added metagram header primary linear gradient. * feat: added flash icon. * feat: added secondary state of header. * feat: added secondary state of header with menu. * chore: cleaned some code. * docs: updated component docs. --------- Co-authored-by: SoSweetHam <[email protected]> * Feat/metagram message (#135) * feat: added metagram message component. * feat: added both states of message component. * docs: added usage docs. * chore: exposed component from ui. * fix: component -> fragement --------- Co-authored-by: SoSweetHam <[email protected]> * feat: modal (#154) * fix: styling of modal * fix: modal props * fix: conflicting styles * fix: styles of drawer * fix: hide scrollbar in drawer * fix: padding * fix: used native method for dismissing of drawer * feat: Context-Menu (#156) * feat: Context-Menu * fix: name of component * fix: as per suggestion * fix: action menu position * fix: class * feat: responsive-setup (#157) * feat: responsive-setup * fix: background color * fix: added font fmaily * feat: responsive setup for mobile and desktop (#159) * feat: responsive setup for mobile and desktop * fix: width of sidebar and rightaside * fix: responsive layout * feat: SideBar * fix: added some finishing touches to sidebar and button * fix: prevent pages transition on desktop * fix: icon center * feat: settings page and icon added * feat/layout-enhancement (#168) * feat/infinite-scroll (#170) * feat/infinite-scroll * fix: aspect ratio of post * fix: bottom nav background * settings page (#169) * settings page layout done * settings page layout done * formt fix * format fix * format fix * routing for settings page fixed * settings page buttons * merge conflict * settings page tertiary pages * settings pages all done * settings pages unnecessary page deleted * requested changes done * requested changes done * Feat/comments pane (#171) * feat/comments-pane * fix: overflow and drawer swipe * feat: Comment fragment * fix: comments added * fix: comment fragment * feat: Comments reply * fix: message input position * fix: post type shifted to types file * fix: one level deep only * fix: drawer should only be render on mobile * fix: comments on layout page * fix: format * feat: messages (#174) * feat: messages * feat: ChatMessae * feat: messages by id * fix: messages page * fix: icon name * fix: hide bottom nav for chat * fix: header * fix: message bubble * fix: message bubble * fix: message bubble * fix: as per suggestion * fix: messaging * chore: change from nomad to k8s (#179) * chore: change from nomad to k8s * Update infrastructure/eid-wallet/src/routes/+layout.svelte Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * feat: uri extraction * feat: regitry stuff * feat: registry using local db * 📝 Add docstrings to `feat/switch-to-k8s` (#181) Docstrings generation was requested by @coodos. * #179 (comment) The following files were modified: * `infrastructure/evault-provisioner/src/templates/evault.nomad.ts` Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * chore: format --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * fix: make scan qr page work again (#185) * feat: Discover Page (#180) * refactor/Post (#186) * refactor/Post * fix: format and lint * fix: added dots for gallery * fix: added dots for gallery * fix: added dots for gallery * fix: plural name * feat: splash-screen (#187) * Feat/evault provisioning via phone (#188) * feat: eid wallet basic ui for verification * chore: evault provisioning * feat: working wallet with provisioning * feat: restrict people on dupes * 📝 Add docstrings to `feat/evault-provisioning-via-phone` (#189) Docstrings generation was requested by @coodos. * #188 (comment) The following files were modified: * `infrastructure/eid-wallet/src/lib/utils/capitalize.ts` * `infrastructure/evault-provisioner/src/utils/hmac.ts` Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * feat: added uploaded post view component. (#182) * feat: added uploaded post view component. * fix: fixed the outline and color. * fix: moved function to external definition. * fix: fixed the restProps. * profile page (#178) * basic layout for profile page * fixed alt text * merge conflict * profile page for other users implemented * fix: profile pages and logics * fixed all the pages of profile * fixed all the pages of profile * fix: format --------- Co-authored-by: gourav <[email protected]> * Feat/radio input (#176) * feat: added a radio button custom * docs: added name option in docs. * chore: cleaned the unnecessary classes and variables for input type radio. * fix: moved input radio to its own component. * fix: keydown events added. * feat: added settings tile component. (#184) * feat: added settings tile component. * chore: fixed the naming convention * chore: renamed callback to onclick * fix: fixed the use of restProps * fix: fixed the unnecessary onclick expose. * fix: fixed the join function params. * Feat/textarea (#194) * chore: removed redundant radio * feat: added textarea. * fix: tabindex * fix: removed type inconsitency. * Feat/mobile upload flow (#193) * fix: header logic in secondary * fix: fixed the text in header in post * feat: trying some hack to get file image input. * feat: added image input on clicking the post bottom nav * chore: got rid of non-required code. * feat: added the logic to get the images from user on clicking post tab. * feat: added store. * feat: added correct conversion of files. * feat: added the correct display of image when uploading. * feat: added settings tile to the post page and fixed the settingsTile component type of currentStatus * feat: added hte correct header for the audience page. * fix: fixed the page transition not happening to audience page. * feat: added audience setting * feat: added store to audience. * chore: removed console log * feat: added post button. * feat: correct button placement * fix: horizontal scroll * fix: positioning of the post button. * fix: protecting post route when no image is selected. * fix: improved type saftey * feat: added memory helper function * feat: added memory cleanup. * Feat/social media platforms (#195) * chore: this part works now wooohooo * chore: stash progress * chore: stash progress * chore: init message data models * feat: different socials * chore: blabsy ready for redesign * Feat/social media platforms (#196) * chore: this part works now wooohooo * chore: stash progress * chore: stash progress * chore: init message data models * feat: different socials * chore: blabsy ready for redesign * chore: add other socials * Feat/blabsy add clone (#198) * chore: clone twitter * feat: custom auth with firebase using w3ds * chore: add chat * feat: chat works with sync * feat: twittex * feat: global schemas * feat: blabsy adapter * refactor: shift some text messages to work on blabsy (#199) * chore: stash progress * chore: stash adapters * chore: stash working extractor * feat: adapter working properly for translating to global with globalIDs * feat: adapter toGlobal pristine * chore: stash * feat: adapter working * chore: stash until global translation from pictique * feat: bi-directional sync prestino * feat: bidir adapters * chore: login redir * chore: swap out for sqlite3 * chore: swap out for sqlite3 * chore: server conf * feat: messages one way * feat: ready to deploy * feat: ready to deploy * chore: auth thing pictique * chore: set adapter to node * chore: fix auth token thingy * chore: auth thing * chore: fix auth token thingy * chore: port for blabsy * feat: provision stuff * feat: provision * feat: provision * feat: provision * chore: fix sync * feat: temporary id thing * chore: android * chore: fix mapper sync * chore: fallback * feat: add error handling on stores * feat: fix issue with posts * chore: fix retry loop * Fix/author details (#229) * fix: author-details * fix: owner-details * fix: author avatar * fix: auth user avatar * fix: error handling * fix: author image in bottom nav --------- Co-authored-by: Merul Dhiman <[email protected]> * Fix/change name (#228) * fix: corrected the name to blabsy * fix: extra shit comming. * fix: fixed the alignment of the display in more to look more like current twitter. * fix: avatars (#226) * fix: avatars * fix: avatar in follow request page * fix: images uploaded shown in user profile * fix: button size * fix: avatar --------- Co-authored-by: Merul Dhiman <[email protected]> * chore: temp fix sync * chore: stash progress * Fix/post context menu (#231) * fix: post-context-menu * fix: user id with post * fix: removed redundant code * fix: images * fix: profile data * fix: profile data * fix: image cover * fix: logout * Fix/wallet text (#234) * changed text as per the request and fixed styling on pages with useless scroll * added settings button in main page which went missing somehow * fix: consistent padding * chore: change tags * feat: change icon * feat: webhook dynamic registry * feat: make camera permission work properlyh * chore: removed all locking mechanism thing from platforms * feat: synchronization works perfectly * feat: fixed everything up * feat: changes * chore: stats fix * chore: fix pictique visual issues * chore: fix cosmetic name issue * feat: fix sync issue * chore: fix logical issue here * chore: add qrcode ename * feat: add packages (#235) * feat: add packages * feat: add sample funcs + docs * fixed the filled color on like icon for liked post (#239) * feat: fake passport name * feat: double confirmation * chore: fix pictique login issue * fix: make no user case redir to login * fix: issues with wallet --------- Co-authored-by: Soham Jaiswal <[email protected]> Co-authored-by: SoSweetHam <[email protected]> Co-authored-by: Gourav Saini <[email protected]> Co-authored-by: Bekiboo <[email protected]> Co-authored-by: Julien <[email protected]> Co-authored-by: Ananya Rana <[email protected]> Co-authored-by: Sergey <[email protected]> Co-authored-by: Julien Connault <[email protected]> Co-authored-by: Ananya Rana <[email protected]> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Co-authored-by: Sahil Garg <[email protected]> Co-authored-by: Sahil Garg <[email protected]>
Description of change
Changes twitter text to blabsy, removes non functional component flows too
Issue Number
Type of change
How the change has been tested
n/a
Change checklist
Summary by CodeRabbit
New Features
Style
Bug Fixes
Documentation
Chores