feat: charts #6790
Merged
feat: charts #6790
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
chore: Merge release into master
[master] Release 4.30.4
feat: merge release v4.31.0 into master branch
chore: [master] Merge release 4.33.0
build: [master] Release 4.34.1
build: [master] Release 4.35.1 hotfix
build: merge release v4.36.0 into master
chore: merge release v4.37.1 into master
chore: merge release v4.39.0 into master
build: merge release 4.40.0 into master
build: merge Release 4.41.0 into master
build: merge Release 4.42.0 into master
build: merge release 4.43.2 into master
build: merge Release 4.44.0 into master
build: release 4.45.1 hotfix to master
build: merge release 4.46.1 into master
build: merge release v4.47.0 into master
build: merge release 4.48.2 into master
build: merge Release 4.49.1 into master
build: merge 4.50.3 into master
build: merge Release 4.51.0 into master
chore: merge release 4.52.1 into master
chore: merge 4.53.0 into master
chore: merge 4.54.0 into master
build: merge Release 4.56.0 into master
chore: merge 4.57.0 into master
chore: merge release 4.58.2 into master
build: merge Release v5.0.4 into master
chore: merge v5.1.0 into master
# Conflicts: # frontend/package-lock.json # frontend/package.json
… secret key section
…rmSG into hack/eliminate-data
tshuli
reviewed
Nov 20, 2023
| mb="0.5rem" | ||
| align="center" | ||
| > | ||
| There may not have been any responses collected within this date |
There was a problem hiding this comment.
question: why may?
suggestion: There were no responses collected within this date range.
There was a problem hiding this comment.
Great catch. Updated in fe93acf
FRM-1531 Charts V1.1 | getAllEncryptedSubmissionData to query by date range
BE will need to support lt / gt query that will be passed in from FE this request. |
KenLSM
added a commit
that referenced
this pull request
Nov 21, 2023
* feat: charts (#6790) * chore: import react google charts * feat: create skeleton insights page * fix: regex for route matching for insight page * fix: layout of insights page * feat: dummy endpoint for all encrypted data * feat: queries to get all encrypted submission * fix: encrypted model find params * feat: get and decrypted all submission data * feat: get formfields * feat: generate charts * fix: typing of spacing * feat: create mapping for field to charts * feat: some upgrades to format charts * feat:wordcloud * fix: remove excess divider * fix: prefill rating values in bar chart * feat: add question number to chart title * feat: skeleton for changing chart to table * chore: add gstatic charts to csp policy * fix: rating average counting * feat: table toggle mode * feat: use icon button instead of toggle * feat: dummy date picker * fix: better date range picker styling * feat: date range filtering * fix: do not show rating if no values * fix: filtering by date and styling * fix: increase size of charts * fix: do not display wordcloud if no words * fix: alignment of wordcloud title * fix: typing for submission insights dto * refactor: make code more readable * fix: typing of render array * fix: do not randomize color for rating * feat: fix bar graph colours changing on refocus * feat: fix random word cloud movements on re-render * feat: address MR comments * feat: corrected types and fixed lint comments * feat: update average rating to account for division by 0 * feat: refactored filter function to remove redundant date parsing * feat: set max words for word cloud * fix: fe lint issue * refactor: cleanup constants * feat: add growthbook toggle * fix: flickering pie chart tooltip * fix: ordering of frontend/package.json * chore: add utils * feat: add empty insights field * fix: typeerror on admin submission * refactor: rename insights to charts * feat: add beta badge * refactor: secretkeyverification to common component for results and charts tab * fix: table charts ui * fix: charts secretkeyvewrification component * fix: remove stray space between charts and badge on tab title * fix: remove testing flag * chore: update copy for no charts generted * fix: endday not calculated correctly * feat(be): add limit and reverse chrono sort for submissions query * feat(fe): add forced redirect for email charts * chore: update charts supported field for better visual alignment with secret key section * feat: add marketing prompts for charts * chore: add copy for 1000 chart limit * chore: shorten copy * refactor: create daterangepicker helpers * refactor: use helpers from daterangepicker * feat: add no charts prompt * chore: update language to omit implication of uncertainty * fix: number typo * feat: correctly retrieve based on date range * fix: remove incorrect generic * fix: remove unnecessary comment --------- Co-authored-by: Timothee Groleau <timothee.groleau@gmail.com> Co-authored-by: sebastianwzq <sebastian@open.gov.sg> Co-authored-by: Ken <ken@open.gov.sg> Co-authored-by: tshuli <shuli@open.gov.sg> * fix: omit isVisible property from webhook response (#6907) fix: omit isVisible property * fix(markdown): refine regex to handle newlines after indentation groups (#6917) fix: refine regex to handle newlines after indentation groups * chore: bump version to v6.91.0 --------- Co-authored-by: Foo Chi Fa <59867455+foochifa@users.noreply.github.com> Co-authored-by: Timothee Groleau <timothee.groleau@gmail.com> Co-authored-by: sebastianwzq <sebastian@open.gov.sg> Co-authored-by: tshuli <shuli@open.gov.sg> Co-authored-by: wanlingt <56983748+wanlingt@users.noreply.github.com>
This was referenced Nov 21, 2023
Merged
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.
Problem
Many admins have to transfer their form responses over to another analytics tool.
Closes FRM-1525, Closes FRM-1526, Closes FRM-1531
Solution
Show and process simple charts to provide insights on FormSG directly.
Breaking Changes
Features:
chartsBefore & After Screenshots
Announcement Modal
What's new drawer
No result page
Tests
Regression Tests
Ensure viewing of storage form submission results
Deploy Notes
New dependencies:
react-wordcloud: to render wordcloud on reactstopword: to remove stopwords for wordcloudreact-google-charts: wrapper to google chartsNew dev dependencies:
@types/stopword: types forstopword