Skip to content

Comments

3 new cards on the user page#39

Merged
spottseng merged 2 commits intomainfrom
static-user-upload
Feb 23, 2025
Merged

3 new cards on the user page#39
spottseng merged 2 commits intomainfrom
static-user-upload

Conversation

@spottseng
Copy link
Owner

Summary

Provide a brief summary of the changes in this PR and link to the relevant issue(s) if applicable.
This removed the file upload section of the pages and makes the file reads static. The file upload was complicating a lot of things and there was a persistent bug where the page would refresh every 30 seconds or so and lose the uploaded data.

This is the start to redoing the visualizations of the users page. It includes:

  • photos (stored in app/assets)
  • user profile information
  • user location information

For now the photos have to be manually placed in a directory called app/assets, but will add something later on to automate this. This PR also removes the code that was trying to use IP addresses to look up location data. That was incredibly slow and rate limited so will replace that with something different later.

Does this new change introduce a breaking change?

  • Yes
  • No

If this introduces a breaking change:

  1. Describe the impact on the application below.
  2. Update the BREAKING.md file with the breaking change.
  3. Add "BREAKING CHANGE: [...]" to the commit description when merging.
  4. Create a ticket on the GitHub Project board to address the issue later.

Other Information

Any other information that is useful to note: visual changes, dependency changes, etc.

@spottseng spottseng merged commit 5d187a4 into main Feb 23, 2025
1 check passed
@spottseng spottseng deleted the static-user-upload branch February 23, 2025 13:15
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