-
Notifications
You must be signed in to change notification settings - Fork 6
Release: Firebase Analytics Hardening, SEO Meta Tags, and Glass Effects #104
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
Merged
Merged
Changes from 48 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
8203744
refactor(analytics): use Metro platform file resolution for cross-pla…
BillChirico 36dc454
chore(deps): add expo-glass-effect for liquid glass effects
BillChirico 9150485
feat(theme): add glass effect properties to theme system
BillChirico ffa2d70
feat(components): add GlassView with fallback for non-iOS platforms
BillChirico 3c62c4b
test(GlassView): add native glass path tests
BillChirico 3e5f672
fix(GlassView): remove explicit JSX.Element return type
BillChirico e951b5d
fix(eslint): add exception for expo-glass-effect import resolution
BillChirico d4c1fe5
refactor(analytics): rename impl.* to platform.* following React Nati…
BillChirico 423867a
test(analytics): add tests for fallback platform and utility functions
BillChirico bc988a1
fix(journey): center Days Sober display by removing icon
BillChirico 4f43dcc
Update app.config.ts
BillChirico 7739fea
chore(gitignore): remove Firebase config files from .gitignore
BillChirico 8e9561c
refactor(settings): update sign out behavior and improve modal handling
BillChirico a92575e
fix(analytics): make platform functions synchronous to match public API
BillChirico fc1b7ff
fix(analytics): retrieve analytics instance on re-initialization
BillChirico ff5e30d
fix(analytics): add warning for incomplete Firebase config
BillChirico f066f9d
refactor(plugins): improve withModularHeaders robustness
BillChirico 7ed7b18
test(theme): add dark theme tests for glass properties
BillChirico d2788a1
test(analytics): update tests for synchronous fire-and-forget API
BillChirico bcf88e4
fix(analytics): make initialization platform-aware for native builds
BillChirico 09acb40
security(firebase): remove config files from version control
BillChirico b81de4b
feat(firebase): add config plugin for EAS secret injection
BillChirico 0ad364a
feat(firebase): support both file and string EAS secret types
BillChirico 0426c19
refactor(firebase): simplify plugin to use file secrets only
BillChirico 4d5e128
feat(firebase): add firebase.json for analytics configuration
BillChirico 1b8d140
refactor(analytics): migrate native to Firebase v22 modular API
BillChirico d578839
chore: cleanup test utilities and fix config plugin
BillChirico 5190d76
fix(plugin): correct variable reference in withModularHeaders
BillChirico 0c485b1
fix(analytics): use logEvent for screen views to avoid deprecation wa…
BillChirico c0ced95
fix(analytics): defer getAnalytics() call to prevent module-scope crash
BillChirico 78d4575
fix(analytics): enable analytics collection in production, not just d…
BillChirico 7cf2215
fix(ci): prevent duplicate workflow runs on PR pushes
BillChirico c87a6c0
fix(ci): streamline push trigger for main branch
BillChirico f169778
fix(plugin): handle EAS FILE_BASE64 secrets correctly
BillChirico f602734
docs(analytics): add enhanced docstrings for platform modules
coderabbitai[bot] 76012e4
chore: merge main into develop to resolve PR conflicts
BillChirico e9ea511
fix(firebase): use EAS secret paths for googleServicesFile config
BillChirico 9c225ad
fix(analytics): prevent duplicate Firebase connections and add SEO me…
BillChirico 30ef8be
📝 Add docstrings to `develop` (#105)
coderabbitai[bot] decd8b3
chore: merge main into develop to resolve PR conflicts
BillChirico d664023
chore: merge origin/develop - resolve withFirebaseConfig conflict
BillChirico 1b43b8f
refactor(mocks): add __esModule flag for ES module interop
BillChirico 9e7fd73
fix(analytics): add try/catch and improve resetAnalytics docs
BillChirico 6495b2a
fix(analytics): reset flag on failure and clarify config docs
BillChirico 997522b
docs(plugin): clarify secretValue supports raw content in JSDoc
BillChirico ca8ecd4
fix(seo): use absolute URLs for OG and Twitter image meta tags
BillChirico 8e641fc
fix(seo): render meta tags unconditionally for crawlers
BillChirico d591b4f
CodeRabbit Generated Unit Tests: Add comprehensive Jest tests for web…
coderabbitai[bot] e93ab80
fix(analytics): handle circular references in sanitizeParamsForLogging
BillChirico 187dbec
refactor(analytics): use Promise-based initialization to prevent race…
BillChirico b5c4bc7
refactor(analytics): use Promise-based initialization in index.ts
BillChirico a35b9a0
docs(mocks): fix JSDoc type annotations in expoRouterHead mock
BillChirico 025e1b5
fix(analytics): add __resetForTesting to index.ts and update tests
BillChirico b66979a
docs: clarify secretValue JSDoc and simplify mock types
BillChirico e494004
test(analytics): add coverage for concurrent initialization and debug…
BillChirico 2e7da0c
Merge branch 'main' into develop
BillChirico 568980d
fix(analytics): correctly handle shared objects in param sanitization
BillChirico e958a54
chore(test): remove unused variable in analytics test
BillChirico File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| /** | ||
| * Mock for expo-router/head module to prevent ESM parsing errors in Jest tests. | ||
| * This provides a simple passthrough component that renders children. | ||
| */ | ||
|
|
||
| const React = require('react'); | ||
|
|
||
| /** | ||
| * Renders the provided children inside a React Fragment. | ||
| * @param {object} props - Component props. | ||
| * @param {import('react').ReactNode} props.children - Elements to render inside the fragment. | ||
| * @returns {import('react').ReactElement} A React Fragment containing the given children. | ||
| */ | ||
| function Head({ children }) { | ||
| return React.createElement(React.Fragment, null, children); | ||
| } | ||
|
|
||
| module.exports = Head; | ||
| module.exports.default = Head; | ||
|
BillChirico marked this conversation as resolved.
|
||
| module.exports.__esModule = true; | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.