Post-launch feedback - Githubissues

Iridescent-CM / technovation-app

The team submission platform for the Technovation Challenge

https://technovationchallenge.org

GNU General Public License v3.0

7 stars 4 forks source link

Post-launch feedback #4909

Closed dboyer closed 1 week ago

dboyer commented 1 month ago

Running list of issues reported after August 12 launch

[x] If you login as a ChA that does not belong to a chapter, clicking Chapter Ambassador Training will cause an error
[x] Default the ChA admin page (/admin/chapter_ambassadors) to show ChAs for the current season
[x] Docusign webhook could not update document on our end

shaun-technovation commented 4 weeks ago

I'm not sure this is the right place to document this, but just wanted to document it somewhere. We had an error from the Dousign webhook, where the envelope_id for the signed document could not be matched to a document on our platform. This is because the document was sent last season (and therefore wouldn't have a matching document on our side), and signed (maybe by a bot or spammer). This error can be safely ignored, there isn't anything for us to do in this case.

More info is in this Slack thread: https://iridescentlearning.slack.com/archives/C06CHBQ4E/p1723640852064449 https://iridescentlearning.slack.com/archives/C02FXA4HJPQ/p1723594659883659

shaun-technovation commented 4 weeks ago

Default the ChA admin page (/admin/chapter_ambassadors) to show ChAs for the current season

This should be fixed on QA now. A side comment, I wonder if we'll want this for the Chapters admin page too at some point.

dboyer commented 4 weeks ago

Confirming that I see the 2025 filter applied as a default to the ChA admin page.

One note is that it's impossible to remove the season and just view all ChAs. This is also true when searching for participants on other admin pages so may be something we want to address globally at another time.

shaun-technovation commented 4 weeks ago

One note is that it's impossible to remove the season and just view all ChAs. This is also true when searching for participants on other admin pages so may be something we want to address globally at another time.

Ah, good find! That's definitely not ideal. I have a fix for the ChA datagrid (which most likely could also be used for the participants datagrid). It will go to QA after code review and can go out with the next deploy (after the one today).

shaun-technovation commented 3 weeks ago

One note is that it's impossible to remove the season and just view all ChAs. This is also true when searching for participants on other admin pages so may be something we want to address globally at another time.

This should be fixed on QA now.

shaun-technovation commented 3 weeks ago

Also, I addressed everything so far for the this issue. We're a week+ past the launch, any new issues we find can go in a new/separate issue. 👍

dboyer commented 2 weeks ago

Confirming on QA that I could remove the season from the participants page and search across all seasons.

dboyer commented 1 week ago

Confirmed on production that I could remove the year filter and search across seasons.