Skip to content

Conversation

@cmonfortep
Copy link
Contributor

@cmonfortep cmonfortep commented Aug 3, 2022

Task/Issue URL: https://app.asana.com/0/715106103902962/1202697459365558/f

Description

Extracts hardcoded strings used in Common module as String resources

Steps to test this PR

  • Ensure unit tests are green

App TP activity list

  • check activity list renders each tracking event correctly

UI changes

Before After
!(Upload before screenshot) (Upload after screenshot)

Copy link
Collaborator

@aitorvs aitorvs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@cmonfortep cmonfortep merged commit 3d473b8 into develop Aug 3, 2022
@cmonfortep cmonfortep deleted the feature/cristian/apptp/hardcoded_strings branch August 3, 2022 15:49
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.

2 participants