Amplitude Personalize Workshop #41
Merged
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
* WIP - Adding content outline for Braze config steps * WIP - Added template create images * WIP - Added steps content, shrunk Retina imgs * WIP - Create Template updates * WIP - Added Personalize arch diagram * WIP - Updated arch image * WIP - Updated arch section * WIP - Arch overview updates * WIP - Moved template files * WIP - Updated arch dia and numbered lists * WIP - Test numbering fix * WIP - Added users CSV file - Removed .zip from .gitignore * WIP - Added user upload content * WIP - Added segment creation * WIP - Added campaign creation * WIP - Campaign content * WIP - Added email example images + verbiage * Added ELB lookup and response exploration * WIP - Editing sample code for recommendations service test * Service lookups and demo rec query * Sample email image change * Updated to support automated user creation * Removed CSV references * Andys updates * WIP - Adding content outline for Braze config steps * WIP - Added template create images * WIP - Added steps content, shrunk Retina imgs * WIP - Create Template updates * WIP - Added Personalize arch diagram * WIP - Updated arch image * WIP - Updated arch section * WIP - Arch overview updates * WIP - Moved template files * WIP - Updated arch dia and numbered lists * WIP - Test numbering fix * WIP - Added users CSV file - Removed .zip from .gitignore * WIP - Added user upload content * WIP - Added segment creation * WIP - Added campaign creation * WIP - Campaign content * WIP - Added email example images + verbiage * Added ELB lookup and response exploration * WIP - Editing sample code for recommendations service test * Service lookups and demo rec query * Sample email image change * Updated to support automated user creation * Removed CSV references * Andys updates * gitignore fix * Remove local references * Template detail screen shot * jupyter download instructions
james-jory
requested changes
Jun 9, 2020
There was a problem hiding this comment.
This is going to be a great workshop! Some things to look at before it's ready.
- Source files require license statements at the top. See generate_users_json.py for example.
- Add requirements.txt for datagenerator.
- Seems like the datagenerator directory could benefit from a README.
- Is the generate_users_json.py obsolete now? Can datagenerator be used to rebuild just the users JSON file or maybe generate_users_json.py can use datagenerator to create the users?
workshop/3-Experimentation/3.5-Amplitude-Performance-Metrics.ipynb
Outdated
Show resolved
Hide resolved
workshop/3-Experimentation/3.5-Amplitude-Performance-Metrics.ipynb
Outdated
Show resolved
Hide resolved
workshop/3-Experimentation/3.5-Amplitude-Performance-Metrics.ipynb
Outdated
Show resolved
Hide resolved
* Update Welcome page * Welcome notebook arch update
…ark/retail-demo-store into amplitude-personalize-workshop * 'amplitude-personalize-workshop' of github.com:manbearshark/retail-demo-store: Data reference and stage copy update User ID to string Staged Amplitude deploy Search report images and content Adding final steps Fixed SSM query issue Fixed settings image Additional config steps Users file read + SSM config read Fix for file paths Datagenerator library + Amplitude reports Post personalize part I More detail on post-personalize reports Switched config file name Post personalization features Added Amplitude notebook
james-jory
requested changes
Jun 15, 2020
There was a problem hiding this comment.
Minor: at the top of the workshop, should the first reference to "Amplitude" be hyperlinked to their website? What about adding a brief "About Amplitude" section that explains what Amplitude is and does? Some readers may not be familiar with them.
Also, I was surprised to see all the Braze workshop files in this PR too.
Hyperlink in first sentence in the "Create Amplitude Conversion Reports" section of the notebook is malformed. Looks like you have parens and square brackets in wrong places.
james-jory
approved these changes
Jun 19, 2020
daemon-joe
pushed a commit
to Daemon-Solutions/retail-demo-store
that referenced
this pull request
Feb 7, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in 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.
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.