Skip to content

feat(ECOM-43): finish apply screen - #6

Open
moazosama1 wants to merge 9 commits into
developmentfrom
feature/ECOM-43/apply
Open

feat(ECOM-43): finish apply screen#6
moazosama1 wants to merge 9 commits into
developmentfrom
feature/ECOM-43/apply

Conversation

@moazosama1

Copy link
Copy Markdown
Collaborator

What type of PR is this? (check all applicable)

  • [✅] ✨ Feature
  • 🐛 Bug Fix
  • ♻️ Refactor
  • 🚀 Optimization/Performance
  • 📖 Documentation Update
  • [] 🧪 Tests

Description


Related Issues / Tickets

  • Closes #
  • Related to #

How to test this PR

Please provide instructions for testing locally (e.g., commands, devices, simulators, or scenarios to test).

  • Run flutter pub get
  • Run flutter test
  • Manual test steps:
    1. ...
    2. ...
    3. ...

Screenshots / Recordings (if applicable)


UI accessibility checklist

If your PR includes UI changes, please check the following:

  • [✅] Semantic widgets used where possible? (e.g., Semantics, Text, IconButton)
  • Tested with screen readers (TalkBack/VoiceOver)?
  • Proper color contrast?
  • Supports different screen sizes/orientations?

Added/Updated Tests?

  • [✅] ✅ Unit Tests (e.g., business logic, Cubit/Bloc, repositories)
  • [✅] ✅ Widget Tests (e.g., rendering, UI state, widget interactions)
  • ✅ Integration Tests (e.g., full app flow, navigation, API calls)
  • ❌ No, and this is why: please explain
  • ❓ Need help with writing tests

@moazosama1 moazosama1 added the enhancement New feature or request label Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant