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

redirect to the View all Deposits page when no deposit has been created #4326

Merged

Conversation

shiftrightonce
Copy link
Contributor

What's this PR do?

#4297

Screenshots (if appropriate)

What Issues does it Close?

Closes
#4297

What are the relevant tickets?

Any background context you want to provide?

Where should the reviewer start?

How should this be manually tested?

Try navigating to the Edit Deposit Slip page when there are no existing deposits

How should the automated tests treat this?

Questions:

  • Is there a related website / article to substantiate / explain this change?

    • Yes - Link:
    • No
  • Does the development wiki need an update?

    • Yes - Link:
    • No
  • Does the user documentation wiki need an update?

    • Yes - Link:
    • No
  • Does this add new dependencies?

    • Yes
    • No
  • Does this need to add new data to the demo database

    • Yes
    • No

@DawoudIO DawoudIO merged commit 688a0ce into ChurchCRM:master Jun 7, 2018
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