Skip to content

Conversation

btangmu
Copy link
Member

@btangmu btangmu commented Aug 26, 2025

-Revise the table layout and content as proposed in the ticket

-Enable downloading as xlsx instead of csv

-Modernize with new FlaggedItems.vue; extensively revise cldrFlagged.mjs

-Fix formatting of VettingParticipation.vue (minor, should have been done in a previous commit for a different ticket)

CLDR-18808

  • This PR completes the ticket.

ALLOW_MANY_COMMITS=true

-Revise the table layout and content as proposed in the ticket

-Enable downloading as xlsx instead of csv

-Modernize with new FlaggedItems.vue; extensively revise cldrFlagged.mjs

-Fix formatting of VettingParticipation.vue (minor, should have been done in a previous commit for a different ticket)
@btangmu btangmu self-assigned this Aug 26, 2025
-Make the sort key for basic same as for details, that is, locale name + days ago
@btangmu btangmu marked this pull request as ready for review August 27, 2025 16:15
@btangmu btangmu requested review from srl295 and macchiati August 27, 2025 17:02
Copy link
Member

@srl295 srl295 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good upgrades

function addUserInfo(err, args, o) {
if (err || o.entry.err || !o.entry.user) {
function storeResponseData(json) {
// Note: the response json format is no longer very well-suited for how it
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why the mishmash?

error: errorHandler,
};
cldrAjax.sendXhr(xhrArgs);
const FLAGGED_RESPONSE_IS_FAST = true;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

more common terminology might be sync/asynchronous or blocking/non blocking.

@btangmu btangmu merged commit c0beb8a into unicode-org:main Sep 2, 2025
15 checks passed
@btangmu btangmu deleted the t18808_b branch September 2, 2025 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants