Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TIMOB-13360] Exporting mediaURL for later cleanup #3519

Closed
wants to merge 1 commit into from

Conversation

alg
Copy link
Contributor

@alg alg commented Dec 4, 2012

Some privacy-caring apps (as ours) may want to remove temp files after processing. This patch returns the temp mediaURL.

The temp can't be removed in the module as the blob won't be read properly.

@WhichKatieDid
Copy link
Contributor

CR passed and CLI signed. Needs documentation and remerging before FR can happen.

@negupta
Copy link
Contributor

negupta commented Feb 19, 2014

TIMOB-1597 does not appear to be the correct ticket number as that ticket was fixed a long time ago. Can you please provide the correct TIMOB number?

@ingo
Copy link
Contributor

ingo commented Feb 21, 2014

We updated the related ticket to TIMOB-13360.

@ingo
Copy link
Contributor

ingo commented Feb 21, 2014

It doesn't appear that @alg will be able to remerge this pull request, as @BlainHamon asked 9 months ago for the same. In this case I will close the PR, and we can adopt (and credit) the code as approprate.

@ingo ingo closed this Feb 21, 2014
@alg
Copy link
Contributor Author

alg commented Feb 21, 2014

Sure, please do whatever you want with it. I was eager to help with this and two or three other tickets, but since no one was interested in paying even slightest attention to my work, figured you guys have better things to do. No longer care either.

@ingo
Copy link
Contributor

ingo commented Feb 21, 2014

@alg, I'm sorry we weren't able to address this quickly in previous versions. We will make sure you are credited for your work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants