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

Remove heatmap #62

Merged
merged 18 commits into from
Sep 8, 2024
Merged

Remove heatmap #62

merged 18 commits into from
Sep 8, 2024

Conversation

drewvolz
Copy link
Owner

@drewvolz drewvolz commented Sep 8, 2024

Largest change

  • Removed heatmap view since bottomless AES encrypted that data
    If you feel so inclined to bring this back, you can investigate:
    • rendered: bottomless.com/data
    • api: /users/clean-data
    • static: static/js/pages/Data/Data.js
      • I gave a shot at unraveling this in Swift using a bridging header with imports for CommonCrypto, added it to the build path, and wrote something to take the encrypted data in the clean-data api response and have common crypto unravel it. Stopping here as it would honestly be faster to parse the data from the webpage.
    let REACT_APP_STATS_KEY = "cmV0dXJuIG5hdmlnYXRvci51c2VyQWdlbnQgKyB3aW5kb3cubG9jYXRpb24ub3JpZ2lu"
    const key = atob(REACT_APP_STATS_KEY) // navigator.userAgent + window.location.origin
    JSON.parse(CryptoJS.AES.decrypt(cleanDataRaw.data, key).toString(CryptoJS.enc.Utf8))

Other changes

  • Bubbled up login errors for easier debugging
  • Inlined titles
  • Removed empty data mappings
  • Assigned a bunch of section headers to have subheadline text style
  • Removed the gifting image from the referrals item

Puzzling changes

  • Xcode suddenly forgot what a scheme was so those were recreated
  • XCTest says lists are now collection views

@drewvolz drewvolz enabled auto-merge (squash) September 8, 2024 07:11
@drewvolz drewvolz disabled auto-merge September 8, 2024 07:14
@drewvolz drewvolz enabled auto-merge (squash) September 8, 2024 07:15
@drewvolz drewvolz merged commit 779fd0c into trunk Sep 8, 2024
1 check passed
@drewvolz drewvolz deleted the drew/remove-heatmap branch September 8, 2024 07:17
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.

1 participant