Skip to content

Conversation

@cmaier
Copy link
Contributor

@cmaier cmaier commented Mar 27, 2024

Custom properties can now be added and implemented easier. As an example the subject text edit for the external billing has been make configurable using a CustomProperty.

APPS-1491 (APPS-1497)

How to test?

  • Checkout this branch for an app making use of external billing, limit the subject length and check if it works.

Definition of Done

  • Changelog gepflegt
  • Dokumentation gepflegt
  • Alle Anforderung des Issues sind erfüllt
  • Selbst greviewed (aka Self-Reviews (eng))
  • Review mit Product Owner (Release-Variante o. Minified Build)

App Tests

  • Minified getestet? (aka. Release Build)
  • Environements beachtet (Production/Staging)
  • Unterstützte Sprachen sind getestet
  • Light-/Dark-Mode getestet
  • Edge-Cases getestet
  • Android API Levels wurden berücksichtigt (minSdk?)

Testing

  • Tests geschrieben (aka Unit-Tests, Integration-Tests)
  • Tests lokal laufen gelassen

@cmaier cmaier requested a review from Fabtron March 27, 2024 12:52
@cmaier cmaier self-assigned this Mar 27, 2024
Copy link
Collaborator

@Fabtron Fabtron left a comment

Choose a reason for hiding this comment

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

Works good just some small change as suggestion

@cmaier cmaier requested a review from Fabtron March 28, 2024 11:08
Copy link
Collaborator

@Fabtron Fabtron left a comment

Choose a reason for hiding this comment

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

Nice

@cmaier cmaier merged commit f90d5de into main Mar 28, 2024
@cmaier cmaier deleted the apps-1497-add-project-specific-property-support branch March 28, 2024 11:40
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