Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sample Pull Request Template #1

Open
1 of 19 tasks
mitulmanish opened this issue Aug 14, 2019 · 0 comments
Open
1 of 19 tasks

Sample Pull Request Template #1

mitulmanish opened this issue Aug 14, 2019 · 0 comments

Comments

@mitulmanish
Copy link
Owner

mitulmanish commented Aug 14, 2019

What it does, and why

  • Please briefly describe what this change does and the motivation behind the change

Please specify the type of change

Please select one:

  • New Feature
  • Bug Fix
  • Refactoring
  • Added tests for existing functionality
  • Updated assets
  • Updated copy(localised text)
  • Updated localised strings using Phrase app pull

Jira Tickets

https://monese.atlassian.net/browse/your-ticket-number-here

Testing

Please select one:

  • Have you added Unit tests
  • I Didn't add Unit tests because _____________
  • Please mention the steps required to test this change
    • Example:
      • Launch the app
      • Enable the feature flag
      • Go to the Cards tab
      • Make sure the CTA button has a green background color

UI Changes

  • If you have developed a new shiny✨ feature, please add a screenshot of the feature.The screenshot could be taken from any device, also attach the link of the design on Zeplin.
  • If you have fixed a bug, then please attach a Before and After screenshots, so that it's easy for the reviewers to spot the changes. Please add your Before and After screenshots using the table below, so they can be viewed side by side.
  • Please use the template below 👇🏼 to arrange your screenshots in a table
    |Before|After|
    |---|---|
    |image-url-here|image-url-here|

Feature Flag 🚩

  • Have you added information regarding enabling/disabling feature flag/flags

UI

Did you test this feature on an actual device or simulator ?

  • Physical Device
  • Simulator

Which iOS version does your feature support ?

  • iOS 10
  • iOS 11
  • iOS 12
  • iOS 13

Constrained related warnings

  • Make sure any new UI you have created does not produce auto layout constraints ambiguity

Localisation 🌍

  • Have you localised the strings ?

Introducing Cross Cutting concerns

  • Have you asked review from Team members who might be affected by your change in this pull request
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

No branches or pull requests

1 participant