0.0.28

- Fix mobile view on user pages
- Fix favicon issue
- Return Artist UUID with scrobble list

.gitlab-ci.yml

@@ -3,7 +3,7 @@ stages:
  - bundle
 
 variables:
-  VERSION: 0.0.27
+  VERSION: 0.0.28
 
 build-go:
   image: golang:1.16.2

docs/changelog.md

@@ -1,3 +1,8 @@
+# 0.0.28
+- Fix mobile view on user pages
+- Fix favicon issue
+- Return Artist UUID with scrobble list
+
 # 0.0.27
 - Navidrome works!
 - Tidy up request/response structure in backend

internal/goscrobble/scrobble.go

@@ -31,7 +31,7 @@ type ScrobbleResponseMeta struct {
 type ScrobbleResponseItem struct {
 	UUID      string            `json:"uuid"`
 	Timestamp time.Time         `json:"time"`
-	Artist    string            `json:"artist"`
+	Artist    ScrobbleTrackItem `json:"artist"`
 	Album     string            `json:"album"`
 	Track     ScrobbleTrackItem `json:"track"`
 	Source    string            `json:"source"`
@@ -67,7 +67,7 @@ func getScrobblesForUser(userUuid string, limit int, page int) (ScrobbleResponse
 	}
 
 	rows, err := db.Query(
-		"SELECT BIN_TO_UUID(`scrobbles`.`uuid`, true), `scrobbles`.`created_at`, GROUP_CONCAT(`artists`.`name` separator ','), `albums`.`name`, BIN_TO_UUID(`tracks`.`uuid`, true), `tracks`.`name`, `scrobbles`.`source` FROM `scrobbles` "+
+		"SELECT BIN_TO_UUID(`scrobbles`.`uuid`, true), `scrobbles`.`created_at`, BIN_TO_UUID(`artists`.`uuid`, true), `artists`.`name`, `albums`.`name`, BIN_TO_UUID(`tracks`.`uuid`, true), `tracks`.`name`, `scrobbles`.`source` FROM `scrobbles` "+
 			"JOIN tracks ON scrobbles.track = tracks.uuid "+
 			"JOIN track_artist ON track_artist.track = tracks.uuid "+
 			"JOIN track_album ON track_album.track = tracks.uuid "+
@@ -87,7 +87,7 @@ func getScrobblesForUser(userUuid string, limit int, page int) (ScrobbleResponse
 
 	for rows.Next() {
 		item := ScrobbleResponseItem{}
-		err := rows.Scan(&item.UUID, &item.Timestamp, &item.Artist, &item.Album, &item.Track.UUID, &item.Track.Name, &item.Source)
+		err := rows.Scan(&item.UUID, &item.Timestamp, &item.Artist.UUID, &item.Artist.Name, &item.Album, &item.Track.UUID, &item.Track.Name, &item.Source)
 		if err != nil {
 			log.Printf("Failed to fetch scrobbles: %+v", err)
 			return scrobbleReq, errors.New("Failed to fetch scrobbles")

internal/goscrobble/server.go

@@ -733,7 +733,7 @@ func getServerInfo(w http.ResponseWriter, r *http.Request) {
 	}
 
 	info := ServerInfo{
-		Version:             "0.0.27",
+		Version:             "0.0.28",
 		RegistrationEnabled: cachedRegistrationEnabled,
 	}
 

internal/goscrobble/track.go

@@ -3,6 +3,7 @@ package goscrobble
 import (
 	"database/sql"
 	"errors"
+	"fmt"
 	"log"
 	"strings"
 )
@@ -165,10 +166,15 @@ func (track *Track) updateTrack(col string, val string, tx *sql.Tx) error {
 
 func getTrackByUUID(uuid string) (Track, error) {
 	var track Track
-	err := db.QueryRow("SELECT BIN_TO_UUID(`uuid`, true), `name`, IFNULL(`desc`,''), IFNULL(`img`,''), `length`, `mbid`, `spotify_id` FROM `tracks` WHERE `uuid` = UUID_TO_BIN(?, true)",
+	err := db.QueryRow("SELECT BIN_TO_UUID(`tracks`.`uuid`, true), `tracks`.`name`, IFNULL(`albums`.`desc`,''), IFNULL(`albums`.`img`,''), `tracks`.`length`, `tracks`.`mbid`, `tracks`.`spotify_id` "+
+		"FROM `tracks` "+
+		"LEFT JOIN track_album ON track_album.track = tracks.uuid "+
+		"LEFT JOIN albums ON track_album.album = albums.uuid "+
+		"WHERE `tracks`.`uuid` = UUID_TO_BIN(?, true)",
 		uuid).Scan(&track.UUID, &track.Name, &track.Desc, &track.Img, &track.Length, &track.MusicBrainzID, &track.SpotifyID)
 
 	if err != nil {
+		fmt.Println(err)
 		return track, errors.New("Invalid UUID")
 	}
 

web/src/Components/ScrobbleTable.js

@@ -3,37 +3,32 @@ import { Link } from 'react-router-dom';
 
 const ScrobbleTable = (props) => {
     return (
-        <div style={{width: `100%`, maxWidth: `900px`}}>
-        <table style={{width: `100%`}} border={1} cellPadding={5}>
-           <thead>
-              <tr>
-                <td>Timestamp</td>
-                <td>Track</td>
-                <td>Artist</td>
-                <td>Album</td>
-              </tr>
-           </thead>
-           <tbody>
+        <div style={{
+         border: `1px solid #FFFFFF`,
+         width: `100%`,
+         display: `flex`,
+         flexWrap: `wrap`,
+         minWidth: `300px`,
+         maxWidth: `900px`,
+        }}>
          {
             props.data &&
             props.data.map(function (element) {
                let localTime = new Date(element.time);
-                     return <tr key={element.uuid}>
-                        <td>{localTime.toLocaleString()}</td>
-                        <td>
+               return <div style={{borderBottom: `1px solid #CCC`, width: `100%`, padding: `2px`}} key={"box" + element.time}>
+                    {localTime.toLocaleString()}<br/>
                     <Link
-                               key={element.track.uuid}
+                        key={"artist" + element.time}
+                        to={"/artist/"+element.artist.uuid}
+                     >{element.artist.name}</Link> -
+                     <Link
+                        key={"track" + element.time}
                         to={"/track/"+element.track.uuid}
-                               >{element.track.name}
-                           </Link>
-                        </td>
-                        <td>{element.artist}</td>
-                        <td>{element.album}</td>
-                     </tr>;
+                     > {element.track.name}</Link>
+               </div>;
+
             })
          }
-           </tbody>
-        </table>
       </div>
     );
 }

web/src/Components/TopTable.css

@@ -1,6 +1,8 @@
 .biggestWrapper {
     display: flex;
     flex-wrap: wrap;
+    align-items: center;
+    justify-content: center;
 }
 
 .biggestBox {

web/src/Components/TopTable.js

@@ -12,7 +12,7 @@ const TopTable = (props) => {
    let tracks = props.items;
 
     return (
-      <div>
+      <div style={{textAlign: `center`}}>
          <span>Top {props.type}s</span>
          <div className="biggestWrapper">
             <div className="biggestBox">

web/src/Pages/Album.js

@@ -49,8 +49,8 @@ const Album = (route) => {
         {album.name}
       </h1>
       <div className="pageBody">
-        MusicBrainzId: {album.mbid}<br/>
-        SpotifyID: {album.spotify_id}
+        {album.mbid && <a rel="noreferrer" target="_blank" href={"https://musicbrainz.org/album/" + album.mbid}>Open on MusicBrainz<br/></a>}
+        {album.spotify_id && <a rel="noreferrer" target="_blank" href={"https://open.spotify.com/album/" + album.spotify_id}>Open on Spotify<br/></a>}
       </div>
     </div>
   );

web/src/Pages/Artist.js

@@ -48,9 +48,10 @@ const Artist = (route) => {
       <h1>
         {artist.name}
       </h1>
-      <div className="pageBody">
-        MusicBrainzId: {artist.mbid}<br/>
-        SpotifyID: {artist.spotify_id}
+      <div className="pageBody" style={{textAlign: `center`}}>
+        <img src={artist.img} alt={artist.name} style={{maxWidth: `300px`, maxHeight: `300px`}}/><br/><br/>
+        {artist.mbid && <a rel="noreferrer" target="_blank" href={"https://musicbrainz.org/artist/" + artist.mbid}>Open on MusicBrainz<br/></a>}
+        {artist.spotify_id && <a rel="noreferrer" target="_blank" href={"https://open.spotify.com/artist/" + artist.spotify_id}>Open on Spotify<br/></a>}
       </div>
     </div>
   );

web/src/Pages/Track.js

@@ -43,6 +43,7 @@ const Track = (route) => {
     )
   }
 
+  console.log(track)
   let length = "0";
   if (track.length && track.length !== '') {
     length = new Date(track.length * 1000).toISOString().substr(11, 8)
@@ -54,9 +55,11 @@ const Track = (route) => {
       <h1>
         {track.name}
       </h1>
-      <div className="pageBody">
-        MusicBrainzId: {track.mbid}<br/>
-        SpotifyID: {track.spotify_id}<br/>
+
+      <div className="pageBody" style={{textAlign: `center`}}>
+        <img src={track.img} alt={track.name} style={{maxWidth: `300px`, maxHeight: `300px`}}/><br/><br/>
+        {track.mbid && <a rel="noreferrer" target="_blank" href={"https://musicbrainz.org/track/" + track.mbid}>Open on MusicBrainz<br/></a>}
+        {track.spotify_id && <a rel="noreferrer" target="_blank" href={"https://open.spotify.com/track/" + track.spotify_id}>Open on Spotify<br/></a>}
         Track Length: {length && length}
       </div>
     </div>