Skip to content
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

New folder arch #386

Merged
merged 5 commits into from
Nov 12, 2024
Merged

New folder arch #386

merged 5 commits into from
Nov 12, 2024

Conversation

yjose
Copy link
Member

@yjose yjose commented Nov 11, 2024

What does this do?

Update the folder structure to include a new lib folder and move the ui folder inside the components folder.

Why did you do this?

This update is an important step toward aligning our project structure with common practices in frontend development. It will make integrating libraries such as rn-reusable much easier.

Aligning our folder structure with common frontend practices will help people feel comfortable with the project.

Who/what does this impact?

Not that much , maybe making the upgrade a bit harder.

How did you test this?

Locally

Copy link

netlify bot commented Nov 11, 2024

Deploy Preview for obytes-starter canceled.

Name Link
🔨 Latest commit 775265f
🔍 Latest deploy log https://app.netlify.com/sites/obytes-starter/deploys/67322f32dd9aa200089c39cb

Copy link
Contributor

github-actions bot commented Nov 11, 2024

💯 Test Coverage

Lines Statements Branches Functions
Coverage: 40%
39.41% (188/477) 34.09% (75/220) 26.48% (49/185)

😎 Tests Results

Tests Skipped Failures Errors Time
40 0 💤 0 ❌ 0 🔥 25.413s ⏱️
👀 Tests Details • (40%)
File% Stmts% Branch% Funcs% LinesUncovered Line #s
All files39.4134.0926.4840.17 
app0000 
   [...messing].tsx0100006
   _layout.tsx000019–61
   login.tsx01000010–18
   onboarding.tsx01000014–48
app/(app)0000 
   _layout.tsx000014–68
   index.tsx000010–29
   settings.tsx000020–24
   style.tsx01000010
app/feed0000 
   [id].tsx000013–39
   add-post.tsx01000016–48
components10.2533.335.8810.52 
   buttons.tsx0100007–8
   card.tsx0100009–18
   colors.tsx00009–41
   inputs.tsx0100009–70
   login-form.tsx10010050100 
   title.tsx0100008–9
   typography.tsx0100007–8
components/settings0000 
   item.tsx000014–16
   items-container.tsx000011–12
   language-item.tsx01000011–35
   theme-item.tsx01000010–36
components/ui71.0352.5562.570.39 
   button.tsx10087.5100100131–135
   checkbox.tsx10056.8110010034, 74–110, 131–171, 189–236, 254
   focus-aware-status-bar.tsx16.660016.668–13
   image.tsx60006018, 29
   index.tsx100100100100 
   input.tsx10085.7110010094, 116
   list.tsx60006014, 29
   modal-keyboard-aware-scroll-view.tsx0100100028–37
   modal.tsx75.755057.147589, 118–121, 130, 144, 158, 176
   progress-bar.tsx12.50012.522–45
   select.tsx77.1453.8471.4276.47183, 233–243, 254
   text.tsx1004010010015–42
   utils.tsx29.160029.1613–16, 26, 34–54
components/ui/icons52.1708.3352.17 
   arrow-right.tsx5000509
   caret-down.tsx100100100100 
   feed.tsx5000506
   github.tsx5000508
   home.tsx00006
   index.tsx0000 
   language.tsx501000506
   rate.tsx5000508
   settings.tsx5000506
   share.tsx5000508
   style.tsx501000506
   support.tsx50005011
   website.tsx50005011
lib53.33038.4661.53 
   index.tsx0000 
   storage.tsx2000206–15
   test-utils.tsx100100100100 
   use-theme-config.tsx000010–36
   utils.ts6002085.715
lib/auth4001050 
   index.tsx33.33014.2835.7119–32
   utils.tsx57.141000100 
lib/hooks250025 
   index.tsx0000 
   use-is-first-time.tsx33.330033.338–12
   use-selected-theme.tsx21.420021.4217–29, 33–36
lib/i18n37.9311.7616.6638.46 
   index.tsx100100100100 
   react-i18next.d.ts0000 
   resources.ts100100100100 
   types.ts0000 
   utils.tsx25016.6623.822–24, 28–38, 43–53

@yjose yjose merged commit c1c49e2 into master Nov 12, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant