Steven R. Loomis
Steven R. Loomis
> Maybe make fetchErr.value explicit where fetchErr is referenced near top of DashboardWidget.vue? And/or add a comment. We should test carefully after merging since there have been changes to these...
> What does ! mean in the Locales field? Is that a magic character? Magic for now but could be a button in the UI.
> Could you for now just add a text note below the box that has the magic > > symbols and what they mean? > > > > On Tue,...
> Optional list of locales (like: aa fr zh) (fr implies fr_CA/etc.) (empty for all locales, '!' for non-TC) > A bit too telegraphic. If you haven't merged yet, I...
@btangmu @macchiati this is now ready. Note it is TC locales, not extended submission locales.
> Whoops, misread this as pertaining to the other ticket. Yes, I can confirm that! And apologies, because of how we use merges, this PR had unrelated commits in it...
@btangmu want to review the js / vue ?
UI idea.. the locales box goes away if 'All' or 'DDL' is chosen.
> > UI idea.. the locales box goes away if 'All' or 'DDL' is chosen. Updated as above. Also the Post button is dimmed if body, subject, locales aren't filled...
ok, so it could skip the dev version, and then hash on all of the code. Maybe not worth it.