Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: sitescripts/extensions/bin/createNightlies.py

Issue 29538644: Issue 5642 - Fix Windows Store devbuild upload routine (Closed)
Patch Set: Remove redundant code and simplify formatting Created Sept. 8, 2017, 11:11 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/extensions/bin/createNightlies.py
===================================================================
--- a/sitescripts/extensions/bin/createNightlies.py
+++ b/sitescripts/extensions/bin/createNightlies.py
@@ -618,10 +618,21 @@
submission_id = submission['id']
file_upload_url = submission['fileUploadUrl']
+ # Update submission
+ submission['applicationPackages'][0]['fileStatus'] = 'PendingDelete'
+ submission['applicationPackages'].append(
+ {
Sebastian Noack 2017/09/08 17:30:31 FWIW, it would be OK to put the parenthesis an bra
+ 'fileStatus': 'PendingUpload',
+ 'fileName': os.path.basename(self.path)
tlucas 2017/09/08 11:26:38 nit: Please add a comma to the last line of a mult
+ }
+ )
+
new_submission_path = '{}/{}'.format(submissions_path,
submission_id)
+ new_submission = json.dumps(submission)
- request = urllib2.Request(new_submission_path, None, headers)
+ request = urllib2.Request(new_submission_path, new_submission, headers)
+ request.get_method = lambda: 'PUT'
opener.open(request).close()
self.upload_appx_file_to_windows_store(file_upload_url)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld