Opened 2 years ago

Closed 20 months ago

#5702 closed enhancement (fixed)

Log why theme submissions are blocked

Reported by: poena's profile poena Owned by:
Milestone: Priority: normal
Component: Theme Review Keywords:


It would be good to know the reason why a theme upload is blocked.

Having this statistic data would help find where information to theme authors need to be improved, if a requirement perhaps is too strict, or if the tools need to be updated.

Change History (9)

This ticket was mentioned in Slack in #meta by poena. View the logs.

2 years ago

This ticket was mentioned in Slack in #meta by poena. View the logs.

2 years ago

#3 @dd32
2 years ago

In 11033:

Theme Directory: Log theme uploads (Successful new themes, Failures, and Updates) to slack #themereview-firehose channel.

See #5702.

#4 @dd32
2 years ago

In 11037:

Theme Directory: When logging to Slack, include the Trac changeset for quick reviews.

See #5702.

#5 @dd32
2 years ago

I think we can call this fixed for now?

While these logs aren't as structured as what we'd need to get proper statistics of which checks are failing, I think they probably suffice for now.

We can always add a dashboard view of the errors over time later on if needed.

#6 @dd32
2 years ago

In 11038:

Theme Directory: When logging theme upload failures to slack, use a more visible :x: icon for the message.

Also uses a nicer class name for when a check class is namespaced.

See #5702.

This ticket was mentioned in Slack in #meta by tellyworth. View the logs.

2 years ago

#8 @poena
20 months ago

Yes in my opinion this can be closed as fixed.

#9 @dd32
20 months ago

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.