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

MM-60179: add parser for loading user survey data #1575

Merged
merged 5 commits into from
Aug 19, 2024

Conversation

ifoukarakis
Copy link
Contributor

@ifoukarakis ifoukarakis commented Aug 14, 2024

Summary

  • Add function to load metadata.
  • Add function to load NPS data and validate.
  • Extract and load all data from a zip file.

Ticket Link

https://mattermost.atlassian.net/browse/MM-60179

@ifoukarakis ifoukarakis added Do Not Merge Should not be merged until this label is removed Work In Progress Not yet ready for review labels Aug 14, 2024
@ifoukarakis ifoukarakis force-pushed the MM-60179-nps-parser branch 2 times, most recently from e61b732 to f5274e2 Compare August 16, 2024 13:36
@ifoukarakis ifoukarakis marked this pull request as ready for review August 16, 2024 13:38
@ifoukarakis ifoukarakis added 2: Dev Review Requires review by a core committer and removed Do Not Merge Should not be merged until this label is removed Work In Progress Not yet ready for review labels Aug 16, 2024
# WHEN: attempt to load a valid user survey package
metadata, survey = load_user_survey_package(SURVEY_DIR / 'valid.zip')

# THEN: expect medata to be loaded correctly
Copy link
Contributor

Choose a reason for hiding this comment

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

medata->metadata

@ifoukarakis ifoukarakis merged commit b4eea78 into master Aug 19, 2024
4 checks passed
@ifoukarakis ifoukarakis deleted the MM-60179-nps-parser branch August 19, 2024 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2: Dev Review Requires review by a core committer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants