[NT-876] Pop ProjectPamphlet to root after project is backed #1075
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cdolm92
approved these changes
Feb 20, 2020
justinswart
added a commit
that referenced
this pull request
Feb 21, 2020
* Add pledge CTA to project description * Update tests * Add goToRewards to ProjectDescriptionViewController * Tidy up * Tidy up * Fix line lengths * [NT-876] Pop ProjectPamphlet to root after project is backed (#1075) * Pop project pamphlet to root VC after project is backed * Alphabetize
justinswart
added a commit
that referenced
this pull request
Mar 4, 2020
* [NT-878, NT-873] Add Experiment and Read More is a full-width button (#1073) * read more button experiment * testing experiment outputs * swift format * naming changes * new snapshots * feedback * swift format * corrected typo * [NT-876] Add Pledge CTA to ProjectDescriptionViewController (#1072) * Add pledge CTA to project description * Update tests * Add goToRewards to ProjectDescriptionViewController * Tidy up * Tidy up * Fix line lengths * [NT-876] Pop ProjectPamphlet to root after project is backed (#1075) * Pop project pamphlet to root VC after project is backed * Alphabetize * [NT-876] Add Project Campaign Experiment Tests (#1076) * Add tests * Fix tests * Remove duplicate * Hide the back this project button to creators of their own projects (#1078) * [NT-877] Project page Optimizely Events (#1079) * optimizely tracking events * swiftlint fix * swift format * Add loader to read more button in second variant (#1077) Co-authored-by: Christella <cdolm92@gmail.com>
ifbarrera
pushed a commit
that referenced
this pull request
Mar 18, 2020
* [NT-878, NT-873] Add Experiment and Read More is a full-width button (#1073) * [NT-876] Add Pledge CTA to ProjectDescriptionViewController (#1072) * [NT-876] Pop ProjectPamphlet to root after project is backed (#1075) * [NT-876] Add Project Campaign Experiment Tests (#1076) * Hide the back this project button to creators of their own projects (#1078) * [NT-877] Project page Optimizely Events (#1079) * Add loader to read more button in second variant (#1077) * [NT-930] Add Project Creator Details Query and VM outputs (#1086) * [NT-940] Activate Creator Details Experiment (#1088) * [NT-931] Add creator byline view (#1089) Co-authored-by: Justin Swart <justinswart@users.noreply.github.com> * [NT- 956] CreatorByline View Model and Tests (#1095) * [NT-937] Creator Details Shimmer Loading View (#1097) * [NT- 939] Creator details tracking events (#1098) * translated copy (#1111) Co-authored-by: Christella <cdolm92@gmail.com>
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📲 What
Pops the
ProjectPamphletViewController's navigation stack to root after a project is backed.Note: This is based off #1072's branch.
🤔 Why
Now that we are presenting the checkout flow from the project campaign details in #1072, once the thanks page is presented and dismissed we want the user to land back on the project page and not the campaign details page. If they exit out of the checkout flow earlier than completion then we do want them to return to the campaign details page.
🛠 How
We were already observing the
projectBackednotification in order to refresh the project after backing. We've now repurposed this signal to pop the navigation stack.👀 See
✅ Acceptance criteria