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

feat: #1127 As a developer, I should be able to see a preview of my app #1431

Merged
merged 6 commits into from
Jun 3, 2020

Conversation

ant066
Copy link
Contributor

@ant066 ant066 commented May 29, 2020

Pull request checklist

Does this close any currently open issues?

fixes: #1127

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • Build (yarn build) was run locally and any changes were pushed
  • Lint (yarn lint) has passed locally and any fixes were made for failures
  • Test (yarn test) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Can not see preview when edit app

Issue Number: #1127

What is the new behavior?

Create new route to preview the app while editing
Screen Shot 2020-05-29 at 10 58 28 AM

Does this introduce a breaking change?

  • Yes
  • No

Other information

@ant066 ant066 added the do not merge Branch should not be merged into main workflow in current state label May 29, 2020
Copy link
Contributor

@willmcvay willmcvay left a comment

Choose a reason for hiding this comment

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

Just missing a couple of tests, otherwise looks good

@duong-se duong-se removed the do not merge Branch should not be merged into main workflow in current state label Jun 2, 2020
@github-actions github-actions bot requested review from duong-se and willmcvay June 2, 2020 08:04
@github-actions github-actions bot added aml-checklist Relates to Anti-money laundering app geo-diary Relates to GEO Diary app lifetime-legal Relates to Lifetime Legal App marketplace Relates to the Marketplace labels Jun 2, 2020
Copy link
Contributor

@duong-se duong-se left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot requested a review from duong-se June 3, 2020 06:54
Copy link
Contributor

@willmcvay willmcvay left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@ant066 ant066 merged commit 705a747 into master Jun 3, 2020
@ant066 ant066 deleted the feat/1127-developer-preview-app branch June 3, 2020 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aml-checklist Relates to Anti-money laundering app geo-diary Relates to GEO Diary app lifetime-legal Relates to Lifetime Legal App marketplace Relates to the Marketplace
Projects
None yet
Development

Successfully merging this pull request may close these issues.

As a developer, I should be able to see a preview of my app listing
3 participants