Skip to content

Commit

Permalink
Mobile: Secure messaging thread quick fix (#12727)
Browse files Browse the repository at this point in the history
* Add v1 message thread endpoint using new api

* Remove pointless override

* Copy over vcr, remove module from controller, add more thorough tests

* Remove redundant test

* Reset cassettes after tests

* Change name of test

* Debug failures

* Relocate cassette and uncomment test

* Add back .to_i for params[:id]

---------

Co-authored-by: Andrew Herzberg <andrew.herzberg@adhocteam.us>
  • Loading branch information
Tonksthebear and aherzberg authored May 18, 2023
1 parent aa2efcb commit 1ee3439
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ module Mobile
module V1
class MessagesController < MessagingController
def thread
message_id = params[:id]
message_id = params[:id].try(:to_i)
resource = client.get_messages_for_thread(message_id)
raise Common::Exceptions::RecordNotFound, message_id if resource.blank?

Expand Down

0 comments on commit 1ee3439

Please sign in to comment.