1003244 - Creation of csv file attachement on new Mozilla Reps Swag Request

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Extensions, defect)

Description

[Konstantina Papadea [:couci]]

Attachment: swagform.csv

In light of our new  collaboration with Staples, I'm requesting the creation of a new csv file attachement at every Mozilla Reps Swag Request. Please use the csv template attached.
I've used {{ when the info needed is been extracted from the fields of the swag form.
On the field on the recipient name field, please use the name that is given in shipping details section.
Let me know if you need any more info.

Thanks in advance

Comment 1

[:glob ✱]

hi :couci,

is this a replacement for the existing xml attachment, or an additional attachment?

Comment 2

[Konstantina Papadea [:couci]]

Hey Bryon,
this is an additional attachment

thanks

Comment 3

[Dylan Hardison [:dylan] (he/him)]

The first column, "Date Required", is that the "Ship Before" date on the form? Also, the last column, "Notes", should that be populated?
My last question: What should go into the "Item" columns? There are 14 of them, do they map to the checkboxes on the swag form or are they just there for padding?

Thanks!

Comment 4

[Konstantina Papadea [:couci]]

Hey Dylan thanks for following up
replies are in line
(In reply to Dylan William Hardison [:dylan] from comment #3)
> The first column, "Date Required", is that the "Ship Before" date on the
> form?
Yes it's the same thing
 Also, the last column, "Notes", should that be populated?
Yes we would like to add there the info from "Addition information for delivery person" on the form
> My last question: What should go into the "Item" columns? There are 14 of
> them, do they map to the checkboxes on the swag form or are they just there
> for padding?
the items columns will be filled in by me manually (we change item codes a lot) so you don't need to worry about them, just leave them blank 
> 
> Thanks!

thanks a lot!

Comment 5

[Dylan Hardison [:dylan] (he/him)]

Alright, form is up and ready to test on bugzilla-dev: https://bugzilla-dev.allizom.org/form:reps:swag

There are no form fields correspending to Mozilla Space, Team, or Department Code. Those are currently hard coded to examples given in the attached csv - is that correct?

Also, there is no "shiptoemail", so both email columns are identical -- that is probably not right. Please test out of the form on https://bugzilla-dev.allizom.org/form:reps:swag and let me know what changes are required.

Thanks!

Comment 6

[Dylan Hardison [:dylan] (he/him)]

Attachment: bug-1003244.patch

Patch for review

Comment 7

[:glob ✱]

Comment on attachment 8438745 [details] [diff] [review]
bug-1003244.patch

let's wait for feedback before reviewing the code.

Comment 8

[Dylan Hardison [:dylan] (he/him)]

It looks like you accidentally cleared the needinfo when the CC list changed? I still need to know the answers for the questions raised in comment #5 :-)

Comment 9

[Konstantina Papadea [:couci]]

Hey Dylan,
fields correspending to Mozilla Space, Team, or Department Code need to be hardcoded as the csv example I sent you. The first e-mail column is also hardcoded while the second e-mail column is the bug's reporter e-mail.
As for the url you've sent me, unfortunately I don't have access.
Thanks for all the help

Comment 10

[Dylan Hardison [:dylan] (he/him)]

Alright, I think everything is set as it should be. I have fixed the group permissions on the bugzilla-dev site so you should be able to try the form now: https://bugzilla-dev.allizom.org/form:reps:swag

Hope it works as expected!

Comment 11

[Konstantina Papadea [:couci]]

Hey Dyla,
thanks for this, one last thing, I've forgotten to mention that the Requester field is also hardcoded (sorry my mistake)
Everything else looks great :)

Comment 12

[Dylan Hardison [:dylan] (he/him)]

Attachment: bug-1003244-final.patch

This is a cleaned up (hopefully de-nitted) patch with the Requester field hard-coded per above comment. I don't really see a need to push to dev but I'm open to the suggestion.

Comment 13

[:glob ✱]

Comment on attachment 8449155 [details] [diff] [review]
bug-1003244-final.patch

Review of attachment 8449155 [details] [diff] [review]:
-----------------------------------------------------------------

r=glob nice

::: extensions/REMO/Extension.pm
@@ +280,5 @@
> +                  description => 'Remo Swag Request (CSV)',
> +                  filename    => 'remo-swag.csv',
> +                  ispatch     => 0,
> +                  isprivate   => 0,
> +                  mimetype    => 'text/csv' });

nit: drop }); down to the next line to match the prior attachment's formatting

@@ +298,1 @@
>              $bug->update($bug->creation_ts);

after this line add:
  delete $bug->{attachments};

this is required to make the attachments visible on the bug.

Comment 14

[Dylan Hardison [:dylan] (he/him)]

To [e-mail]:webtools/bmo/bugzilla.git
   e0d898d..3f631ad  master -> master

Leaving bug unresolved until this is pushed out.

Comment 15

[:glob ✱]

(In reply to Dylan William Hardison [:dylan] from comment #14)
> Leaving bug unresolved until this is pushed out.

we resolve bugs as fixed when they are committed, not when they are deployed.

Comment 16

[:glob ✱]

this is now live.

Comment 17

[Konstantina Papadea [:couci]]

thank you all for this!

Comment 18

[Kelimutu [:kiki]]

Hi all, 

Since our department is change into Participation team, I'm reopening this bug to change the CSV file for the swag request. I'll upload the new CSV file here soon.

Comment 19

[:glob ✱]

(In reply to Rizki D Kelimutu [:kelimutu] from comment #18)
> Since our department is change into Participation team, I'm reopening this
> bug to change the CSV file for the swag request. I'll upload the new CSV
> file here soon.

please file a new bug with the required changes.

thanks.

Comment 20

[Kelimutu [:kiki]]

Attachment: swagform-participation.csv

Here is the new CSV file. Basically, I'm just changing the Requester, Email, Team, and Department Code columns. Let me know if you need any other info. 

Thanks in advance

Comment 21

[Kelimutu [:kiki]]

(In reply to Byron Jones ‹:glob› from comment #19)
> (In reply to Rizki D Kelimutu [:kelimutu] from comment #18)
> > Since our department is change into Participation team, I'm reopening this
> > bug to change the CSV file for the swag request. I'll upload the new CSV
> > file here soon.
> 
> please file a new bug with the required changes.
> 
> thanks.

Aaah. Ok.