Skip to content

Conversation

@Entropy-3
Copy link
Collaborator

Pull Request Title

wallet navigation

What’s this PR about? 🤔

Describe briefly what this Pull Request is doing. Explain the problem it solves or the feature it adds to the project.

  • Description:
  • What did you change? - (e.g., added a new feature, fixed a bug, updated something)

Why are we doing this? 💡

Explain why these changes are necessary.

  • Reason: - (e.g., fixing a bug, improving functionality, adding a new feature)

Type of Change 🛠️

What kind of change does this Pull Request introduce? Check the box that applies.

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 📚 Documentation update (non-code changes, such as readme or comments)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 🧹 Code clean-up/refactoring (basic code cleaning and or variable renaming)

How did you test it? 🧪

Explain how you tested your changes.

  • What did you test?
    • Tested locally
    • Wrote automated tests
    • I ran the code and it didn't break
    • Other: ______

Screenshots (if helpful) 📸

If your changes include something visual, like UI changes, add screenshots here!

Checklist ✅

Make sure you’ve done these things before submitting.

  • Code is running without errors
  • Everything works as expected
  • No unnecessary files or code
  • Documentation is updated if necessary
  • Pull Request title is clear and makes sense
  • All checkboxes are checked off!

Anything else? 💬

Any extra notes, questions, or concerns for your partner or reviewers?

Entropy-3 and others added 4 commits June 9, 2025 22:02
- Changed the text color for negative wallet balances in `WalletMainAdapter` from `R.color.error` to `R.color.expense_red`.
- Changed the action ID for navigating from `WalletMainFragment` to the wallet report screen from `action_walletMainFragment_to_walletReportFragment` to `action_navigation_wallet_main_to_report_navigation_graph`.
- Updated `main_navigation_graph.xml` to reflect this change in the action destination.
@Entropy-3 Entropy-3 requested a review from AlphaSweater June 9, 2025 20:13
@AlphaSweater AlphaSweater self-assigned this Jun 9, 2025
@AlphaSweater AlphaSweater merged commit 46cef6f into development Jun 9, 2025
1 check passed
@AlphaSweater AlphaSweater deleted the feature/wallet/nathan branch June 9, 2025 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants