search

OfficeDev / microsoft-teams-apps-champion-management

Champion Management Platform is a custom Teams app that enables organizations to onboard and maintain champions/ SME in their organization in Teams.
MIT License
130 stars 49 forks source link

Event entry is being added to user's Record automatically with incorrect Count #151

Closed LisaJo48 closed 1 year ago

LisaJo48 commented 1 year ago

When a user decides to join the CMP, I have found that 10 points are automatically given to that user even though they have not attended an event yet. After much investigation I was able to determine that the event in the Event List with the ID# of 1 is automatically logged as an entry in the users Record and they are automatically awarded 10 points.

Therefore, I updated the event in the Event list with the ID# of 1 to show the following: Title = "New Champion" Count = 0

This, I expected, would solve the issue so that, when a new user joins the program, the Event entry added automatically and shown in their Record would be named "New Champion" and would not award any points. However, name of the event shown in the user's Record references the title given in the Event List but the points awarded is not referencing the Count column in the Event List. The Count column, for this ID#1 is acting as if it is hardcoded to 10 somewhere regardless of what the Event List indicates. I want to be able to assign my own points/count for all the events I list in the Events list including that of ID#1.

v-saikirang commented 1 year ago

Hi @LisaJo48 ,

It's actually by design in the initial version of the app that a new user gets 10 points by default. It's hard coded for now. However we have already changed this logic to avoid automatically awarding points to new users. This will be available in our next version for which we do not have a release date yet. For now you could just delete the entry in event details list for new users.

Thanks for your feedback on the app.

LisaJo48 commented 1 year ago

That fix will be very helpful as this is causing some confusion. The sooner the better. Please keep me in the loop for the due date of the next release.

v-saikirang commented 1 year ago

Sure @LisaJo48. Please watch out for any updates in this repo in coming weeks. I am closing this ticket for now. Please let me know if you need more information. Thanks.

LisaJo48 commented 1 year ago

Hello @v-saikirang , It appears as if this has been corrected. I've tested a couple times to find that an event entry is not being added automatically and, therefore of course, the count is no longer incorrect as well.

Can you please verify this? Thanks!

v-saikirang commented 1 year ago

Hi @LisaJo48,

Yes, this has been corrected. Thanks for verifying.

Thanks.

LisaJo48 commented 1 year ago

Excellent. Thanks for the heads up. I can update my workaround to accommodate that change.