martinroger / ipodesp32

MIT License
1 stars 0 forks source link

Missing temporisation of the new track info : waitMetadataUpdate flag is not used correctly #6

Closed APF-MRoger closed 1 month ago

APF-MRoger commented 2 months ago

Either a second flag needs to be set in the espod class to temporise sending out the new metadata after request from the car, or this is something to be handled from the metadata handler.

APF-MRoger commented 1 month ago

Change of mechanism based on setting a pending ack when changing track from the radio. Only once metadata is received on four ids (without really checking before/after difference) is the new track info ready to be sent to the Mini (which should respect the requirement for the cmd pending ack). waitMetadataUpdate flag is now deprecated and trackChangeAckPending is introduced as a byte-flag instead of a bool-flag. Commit ad4c92e should be first test version of the fix that follows a harmonised approach.

martinroger commented 1 month ago

69a8fa6 fixes this issue definitively.