Skip to content

Small fix for app snapshots#1538

Merged
stark4n6 merged 3 commits into
abrignoni:mainfrom
maala-nfi:smallFixForAppSnapshots
Jun 19, 2026
Merged

Small fix for app snapshots#1538
stark4n6 merged 3 commits into
abrignoni:mainfrom
maala-nfi:smallFixForAppSnapshots

Conversation

@maala-nfi

Copy link
Copy Markdown
Contributor

Since _lava_media_info.updated_at is the second to last column in the result of the query used in function lava_get_full_media_info, the list item used for variable last_modified_date should be updated from -1 to -2. (The query was updated in commit 28ea00e.)

@JamesHabben

Copy link
Copy Markdown
Collaborator

Good catch on updated_at moving positions.

Could we avoid the positional index here and use the row column name instead?

@JamesHabben JamesHabben added the question Further information is requested label Jun 18, 2026
@maala-nfi

Copy link
Copy Markdown
Contributor Author

Could we avoid the positional index here and use the row column name instead?

Yes! That would be even better.
I tested it and committed the change.

@stark4n6 stark4n6 merged commit 32c0719 into abrignoni:main Jun 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants