Fix-9127: Add the unique ticket code into the downlad CSV file

[ghost]

Fixes #https://github.com/fossasia/open-event-server/issues/9127

Short description of what this resolves:

• Add the unique ticket code into the downlad CSV file

Changes proposed in this pull request:

• Add the unique ticket code into the downlad CSV file

Checklist

• [x] I have read the Contribution & Best practices Guide.
• [x] My branch is up-to-date with the Upstream development branch.
• [ ] The acceptance, integration, unit tests and linter pass locally with my changes
• [ ] I have added tests that prove my fix is effective or that my feature works
• [ ] I have added necessary documentation (if appropriate)

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
open-event-frontend	❌ Failed (Inspect)			Mar 26, 2024 7:43am

[ghost]

After I tried to figure out the problem, the UI crashed on the 'View attendees' screen. I see 2 problems: 1/ If the order has 2 or more tickets, the UI will not display the "Order Id" price. 2/ The "View attendees" screen has 2 columns "Ticket number". I checked the source code and found out this problem happened 7 months ago.

After fixing the above two issues, at "View Attendees" I noticed that the "Order" Column is not the "Ticket ID" in the QR and CSV. Do you want to change the value of the "Orders" column?

[norbusan]

@lthanhhieu you cannot merge development, sorry. please:

• checkout development-backup20240326 branch
• make a diff from that branch to your code
• only include the diff into a new PR.

I just did this and it showed far less changes:

diff --git a/app/components/public/stream/video-stream.ts b/app/components/public/stream/video-stream.ts
index 4553e7ac..dddea07c 100644
--- a/app/components/public/stream/video-stream.ts
+++ b/app/components/public/stream/video-stream.ts
@@ -166,7 +166,6 @@ export default class PublicStreamVideoStream extends Component<Args> {
   async setupRoomChat(stream:any) {
     this.currentRoom = stream;
     this.shown = false;
-    this.eventCheckIn(this.args.event.identifier, stream.microlocationId)
   }

   @action
diff --git a/app/controllers/events/view/tickets/attendees/list.js b/app/controllers/events/view/tickets/attendees/list.js
index b17ae99d..4f595b4d 100644
--- a/app/controllers/events/view/tickets/attendees/list.js
+++ b/app/controllers/events/view/tickets/attendees/list.js
@@ -42,13 +42,6 @@ export default class extends Controller.extend(EmberTableControllerMixin) {
         headerComponent : 'tables/headers/sort',
         isSortable      : true
       },
-      {
-        name            : 'Ticket Name',
-        width           : 80,
-        valuePath       : 'ticket.name',
-        headerComponent : 'tables/headers/sort',
-        isSortable      : true
-      },
       {
         name            : 'Tags',
         width           : 100,
diff --git a/app/routes/events/view/tickets/attendees/list.js b/app/routes/events/view/tickets/attendees/list.js
index 986db7d6..2a210efe 100644
--- a/app/routes/events/view/tickets/attendees/list.js
+++ b/app/routes/events/view/tickets/attendees/list.js
@@ -122,7 +122,7 @@ export default class extends Route.extend(EmberTableRouteMixin) {

     let queryString = {
-      include        : 'user,order',
+      include        : 'user,ticket',
       filter         : filterOptions,
       'page[size]'   : params.per_page,
       'page[number]' : params.page || 1
@@ -134,9 +134,11 @@ export default class extends Route.extend(EmberTableRouteMixin) {
       this.addDefaultValue(tags);
     }
     const attendees = await this.asArray(await eventDetails.query('attendees', queryString));
+    const orders = await this.asArray(await eventDetails.query('orders', {}));
     return {
       tags,
-      attendees
+      attendees,
+      orders
     };
   }
 }

The reason is that we had to force push development, and your PR would bring in all the other stuff again.

[norbusan]

In the above diff I sent, I am NOT sure that the removal of the line

this.eventCheckIn(this.args.event.identifier, stream.microlocationId)

is correct.