tbrittain / vault-bot Goto Github PK

Would be a better visual representation of the song characteristics.
Should show:

1. Song characteristics for that given song
2. Average song characteristics for all songs
3. Average song characteristics for that artist

Some places in the Discord app use UTC time, while the database was recently set to use Central (UTC-6)

Type 'IResolvers<unknown, unknown, Record<string, any>, any>' is not assignable to type 'IResolvers<any, any> | IResolvers<any, any>[]'.
  Type 'IResolvers<unknown, unknown, Record<string, any>, any>' is missing the following properties from type 'IResolvers<any, any>[]': length, pop, push, concat, and 29 more.ts(2322)
types.d.ts(51, 5): The expected type comes from property 'resolvers' which is declared here on type 'ApolloServerExpressConfig'
(property) Config.resolvers?: IResolvers<any, any> | IResolvers<any, any>[]

Probably an issue coming from @graphql-tools/merge or @graphql-tools/load-files

Would be helpful due to audio on desktop devices by default playing at maximum volume, which can be very loud and startling.
Should be perhaps half volume for non-mobile devices and max volume for mobile devices.

Due to Spotify's 10,000 song limit for playlists, the archive playlist will inevitably reach capacity, so the application should keep track of the current archive's capacity and generate a new one once it reaches 10,000 songs

Currently using the following dependencies that require updating to the next major versions:

"@material-ui/core": "^4.11.4",
"@material-ui/data-grid": "*",
"@material-ui/icons": "^4.11.2",
"@material-ui/lab": "^4.0.0-alpha.58"

Current npm package: https://www.npmjs.com/package/@material-ui/core

Packages to update to:
https://www.npmjs.com/package/@mui/material
https://mui.com/components/data-grid/getting-started/#installation
https://mui.com/components/about-the-lab/ (if required, since usages of Lab components appear to have been moved to the main package)

see songs/6U0GYSiAsb9bYvuOzQJLoq

https://stackoverflow.com/questions/70193712/how-to-scroll-to-top-on-route-change-with-react-router-dom-v6

https://en.wikipedia.org/wiki/Test

Takes up full screen rather than only the width of the result container

This will eventually be a problem in a long time, but Spotify's playlist song limit is 10,000. The genre aggregate playlist is the only one without an explicit song limit.

Should focus on #8 before this one.

Make the comparison without the hour remainder so it does not need to be an 8 day maximum difference

/genres/ambeat

Only on mobile, and only with partial entry of search query.
Example: here

Should have better historical data display and more intuitive layout rather than the swipeable cards. Also ideally using D3 rather than Chart.js

More of a long-term problem to tackle, but it could be nice to view the playlists on the site and see their evolution over time

Requirements.txt contains indirect dependencies and can probably be simplified

This is specifically an issue for artists who use their first and last name as their artist name, but their Wikipedia page lists their full name including middle name, so that the first and last names are not directly next to each other.

Should not have the same onHover event for displaying the artist name

Buggy in practice, might not be worth the effort

https://mui.com/customization/dark-mode/#main-content

Example: here

Having them on the same page is not great

https://www.npmjs.com/package//slugify

New tab for songs showing songs that are similar to the given song

Update to Mui v5 broke some Typography component formatting

As the title says. Need to look into why this is occurring and push out an update.
Update: serial port 1 on the VM provided the following error: Docker: invalid reference format.

As per convention

https://reactjs.org/docs/hooks-custom.html

Better consistency between browsers, etc. Need to do app-wide.

Since development will use a local copy of the database, the application should no longer simulate actions

https://react-chartjs-2.netlify.app/docs/chartjs-v2

Could be useful for updating artist art

Strip out some functionality that the Discord bot current coordinates in favor of serverless functions that execute from a cron job-like trigger.
Main function types:

1. Aggregate
   a. Generate aggregate playlists and persist them
   b. Calculate averages for existing dynamic playlist data
2. Purge
   a. Remove expired songs from dynamic playlist

Ex: https://gabnotes.org/lighten-your-python-image-docker-multi-stage-builds/

Example: here

Not having any data to work with could cause some issues with the historical tracking, aggregate playlist generation, etc.

Should check that that artist + song is unique. This is a problem with multiple same songs by the same artist being added to playlists because they are in different albums (such as a song as a single + from its album)

Tracking issue for:

• https://github.com/tbrittain/vault-bot/security/code-scanning/5

Not very high severity due to the strings the query receives being from the Wikipedia API. Regardless, replace with some regex

And name queries so they are no longer anonymous

Use the accessibility tags that come built-in with Material UI

https://spotipy.readthedocs.io/en/2.19.0/?highlight=reorder#spotipy.client.Spotify.playlist_reorder_items

Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in a useEffect cleanup function.