Conversation
…co lint + build. Formateo de archivos previos
Rationale: - CI build requires google-services.json to compile - APK built with mock credentials is non-functional (Firebase Auth fails) - No value in preserving broken APK as artifact Current strategy: - CI validates lint + build (compilation succeeds) - Manual distribution from local build with real credentials - Future: Firebase App Distribution if frequent testing needed
Added to .gitignore: - docs/context/ (internal architecture and strategy docs) - docs/plans/ (implementation plans) - docs/spikes/ (spike execution results and logs) - .github/copilot-instructions.md (AI agent instructions) These files remain local for development but won't be pushed to public repo.
Removed from tracking (remain local only): - docs/context/Spikes_Resultados.md - docs/spikes/s10-oauth-validation-checklist.md All spike documentation now private until thesis completion. Decision on public portfolio content deferred to final review.
- Add intent-filter for miempresa://catalogo scheme - Parse sheetId parameter in MainActivity.onCreate() - Validated: ADB, special chars, WhatsApp invocation - Deferred to US-026: navigation logic, Room queries, error handling Results: 6/6 tests passed, 100% success rate. Intent-filter + parsing validated.
- Dummy activity for Spike 7 validation - Tests wa.me URL format with 3 country codes - Validates UTF-8 encoding (ñ, $, €, emoji) - Will be deleted post-testing
- Add activity declaration for Spike 7 - Not exported (internal use only) - Uses app theme
- Validated wa.me API with URLEncoder for international numbers - Tested encoding: emoji, $, €, line breaks - All 3 tests passed (Argentina, USA, Uruguay) - Spike type: DUMMY (code discarded, docs preserved) - Findings documented for US-038 implementation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Apply RESUMED lifecycle guards to user-triggered navigation callbacks in NavHost and protect critical full-screen form actions (save/delete/create) when screens are not interactive.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- centralize route contract in MiEmpresaRoutes\n- add navigation helpers and admin tab metadata\n- remove legacy MiEmpresaScreen route holder\n- align ADR-002 with implementation progress\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- add auth/client/admin graph routes to route contract\n- nest NavHost destinations by domain graph\n- update ADR-002 phase progress\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- centralize session fallback route resolution in NavHost - normalize sign-out exits to authGraph entry - align auth check pop targets with adminGraph - sync ADR-002 progress notes Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- enable Kotlin Serialization for Navigation Compose - migrate SignIn and OrderDetail to typed routes - keep incremental mixed migration strategy - sync ADR-002 phase notes Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- expand type-safe routes across auth/client/admin flows - keep nested domain graphs with reduced backstack coupling - centralize typed route models and trim legacy route contract - mark ADR-002 complete for MVP scope (phase 3 deferred) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- refine public-facing purpose and wording - clarify adaptive phase deferral criteria - keep phase execution status explicit for MVP scope Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Implement explicit domain use cases for cart/checkout decisions and phone normalization, wire them into key viewmodels, and add unit tests. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Generate full C4 architecture documentation using specialized agents: - C4 Code level: core, cart, and feature modules analysis - C4 Component level: 11 components across 3 architectural layers - C4 Container level: 6 deployment containers with API interfaces - C4 Context level: personas, user journeys, external systems - Update architecture README with C4 index table Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…t reactivity - Add 'Ver mi catálogo' ActionCard in Empresa tab below share actions (H-21) Uses SuccessGreen color, Visibility icon, navigates to ClientCatalogRoute - Fix category product count not updating on product deletion (Bug-1) Add observeProductChanges() Flow to CategoriesRepository, combine in ViewModel so product table changes trigger count recalculation - Expose companyId from ConfigViewModel for catalog navigation - Bump version to 2.2.0 (versionCode 9) - Fix outdated navigation references in c4-context.md (drawer → tabs) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
No description provided.