Skip to content

Conversation

@john-bodley
Copy link
Member

@john-bodley john-bodley commented Apr 6, 2023

SUMMARY

The Airbnb brand color palette changed. This PR reflects these changes (per the attached screenshot).

image

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

CI.

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@codecov
Copy link

codecov bot commented Apr 6, 2023

Codecov Report

Merging #23619 (13bf99d) into master (60e1526) will decrease coverage by 10.23%.
Report is 7 commits behind head on master.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #23619       +/-   ##
===========================================
- Coverage   69.00%   58.77%   -10.23%     
===========================================
  Files        1938     1938               
  Lines       75823    75823               
  Branches     8427     8427               
===========================================
- Hits        52321    44566     -7755     
- Misses      21333    29088     +7755     
  Partials     2169     2169               
Flag Coverage Δ
hive 53.66% <ø> (ø)
javascript 56.22% <ø> (ø)
mysql ?
postgres ?
presto 53.61% <ø> (ø)
python 61.52% <ø> (-21.26%) ⬇️
sqlite ?
unit 55.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...-core/src/color/colorSchemes/categorical/airbnb.ts 100.00% <ø> (ø)

... and 293 files with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@john-bodley john-bodley force-pushed the john-bodley--colors-airbnb branch 2 times, most recently from 440c95f to cdf49ec Compare April 7, 2023 06:27
@john-bodley john-bodley changed the title chore(colors): Updating Airbnb branch colors chore(colors): Updating Airbnb brand colors Apr 7, 2023
@john-bodley john-bodley force-pushed the john-bodley--colors-airbnb branch from cdf49ec to c4c9605 Compare April 7, 2023 20:59
@john-bodley john-bodley reopened this Nov 3, 2023
@john-bodley john-bodley force-pushed the john-bodley--colors-airbnb branch from c4c9605 to d263006 Compare November 3, 2023 18:30
@john-bodley john-bodley force-pushed the john-bodley--colors-airbnb branch from d263006 to 9f6909f Compare November 3, 2023 18:33
@john-bodley john-bodley marked this pull request as ready for review November 3, 2023 20:14
@john-bodley
Copy link
Member Author

@justinpark or @michael-s-molina would you remind reviewing this? I dropped the ball on re-updating the color palette after there was alignment on which specific palette to use.

@john-bodley john-bodley force-pushed the john-bodley--colors-airbnb branch from 9f6909f to 13bf99d Compare November 6, 2023 17:13
Copy link
Member

@michael-s-molina michael-s-molina left a comment

Choose a reason for hiding this comment

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

LGTM

@john-bodley
Copy link
Member Author

@rusackas would you mind stamping this as you’re one of the required reviewers.

@john-bodley john-bodley added the v3.0 Label added by the release manager to track PRs to be included in the 3.0 branch label Nov 9, 2023
@john-bodley john-bodley merged commit 6d8424c into master Nov 13, 2023
@rusackas rusackas deleted the john-bodley--colors-airbnb branch November 13, 2023 19:34
michael-s-molina pushed a commit that referenced this pull request Nov 16, 2023
saghatelian added a commit to 10webio/superset that referenced this pull request Nov 27, 2023
* fix(sqllab): reinstate "Force trino client async execution" (apache#25680)

* fix: remove unnecessary redirect (apache#25679)

(cherry picked from commit da42bf2)

* fix(chore): dashboard requests to database equal the number of slices it has (apache#24709)

(cherry picked from commit 75a7431)

* fix: bump to FAB 4.3.9 remove CSP exception (apache#25712)

(cherry picked from commit 8fb0c8d)

* fix(horizontal filter label): show full tooltip with ellipsis (apache#25732)

(cherry picked from commit e4173d9)

* fix: Revert "fix(Charts): Set max row limit + removed the option to use an empty row limit value" (apache#25753)

(cherry picked from commit e2fe967)

* fix: dataset update uniqueness (apache#25756)

(cherry picked from commit c7f8d11)

* fix(sqllab): slow pop datasource query (apache#25741)

(cherry picked from commit 2a2bc82)

* fix: allow for backward compatible errors (apache#25640)

* fix: DB-specific quoting in Jinja macro (apache#25779)

(cherry picked from commit 5659c87)

* fix: Revert "fix: Apply normalization to all dttm columns (apache#25147)" (apache#25801)

* fix: Resolve issue apache#24195 (apache#25804)

(cherry picked from commit 8737a8a)

* fix(SQL field in edit dataset modal): display full sql query (apache#25768)

(cherry picked from commit 1eba712)

* fix(sqllab): infinite fetching status after results are landed (apache#25814)

(cherry picked from commit 3f28eeb)

* fix: Fires onChange when clearing all values of single select (apache#25853)

(cherry picked from commit 8061d5c)

* fix: the temporal x-axis results in a none time_range. (apache#25429)

Co-authored-by: Elizabeth Thompson <[email protected]>
(cherry picked from commit ae619b1)

* fix(table chart): Show Cell Bars correctly apache#25625 (apache#25707)

(cherry picked from commit 916f7bc)

* fix: remove `update_charts_owners` (apache#25843)

* fix(charts): Time grain is None when dataset uses Jinja (apache#25842)

(cherry picked from commit 7536dd1)

* fix: Saving Mixed Chart with dashboard filter applied breaks adhoc_filter_b (apache#25877)

(cherry picked from commit 268c1dc)

* fix: database version field (apache#25898)

(cherry picked from commit 06ffcd2)

* fix: trino cursor (apache#25897)

(cherry picked from commit cdb18e0)

* chore: Updates CHANGELOG.md for 3.0.2

* fix(trino): allow impersonate_user flag to be imported (apache#25872)

Co-authored-by: John Bodley <[email protected]>
(cherry picked from commit 458be8c)

* fix(table): Double percenting ad-hoc percentage metrics (apache#25857)

(cherry picked from commit 784a478)

* fix(sqllab): invalid sanitization on comparison symbol (apache#25903)

(cherry picked from commit 581d3c7)

* fix: update flask-caching to avoid breaking redis cache, solves apache#25339 (apache#25947)

Co-authored-by: Ville Brofeldt <[email protected]>

* fix: always denorm column value before querying values (apache#25919)

* chore(colors): Updating Airbnb brand colors (apache#23619)

(cherry picked from commit 6d8424c)

* fix: naming denomalized to denormalized in helpers.py (apache#25973)

(cherry picked from commit 5def416)

* fix(helm): Restart all related deployments when bootstrap script changed (apache#25703)

* fix(rls): Update text from tables to datasets in RLS modal (apache#25997)

(cherry picked from commit 210f1f8)

* fix: Make Select component fire onChange listener when a selection is pasted in (apache#25993)

(cherry picked from commit 5fccf67)

* fix(explore): redandant force param (apache#25985)

(cherry picked from commit e7a1876)

* chore: Optimize fetching samples logic (apache#25995)

(cherry picked from commit 326ac4a)

* fix(native filters): rendering performance improvement by reduce overrendering (apache#25901)

(cherry picked from commit e1d73d5)

* fix: update FAB to 4.3.10, Azure user info fix (apache#26037)

(cherry picked from commit 628cd34)

* chore: Updates CHANGELOG.md for 3.0.2 (rc2)

---------

Co-authored-by: Rob Moore <[email protected]>
Co-authored-by: Igor Khrol <[email protected]>
Co-authored-by: Stepan <[email protected]>
Co-authored-by: Daniel Vaz Gaspar <[email protected]>
Co-authored-by: Ross Mabbett <[email protected]>
Co-authored-by: Geido <[email protected]>
Co-authored-by: Beto Dealmeida <[email protected]>
Co-authored-by: JUST.in DO IT <[email protected]>
Co-authored-by: Elizabeth Thompson <[email protected]>
Co-authored-by: John Bodley <[email protected]>
Co-authored-by: Michael S. Molina <[email protected]>
Co-authored-by: mapledan <[email protected]>
Co-authored-by: Arko <[email protected]>
Co-authored-by: Antonio Rivero <[email protected]>
Co-authored-by: Kamil Gabryjelski <[email protected]>
Co-authored-by: Michael S. Molina <[email protected]>
Co-authored-by: FGrobelny <[email protected]>
Co-authored-by: Giacomo Barone <[email protected]>
Co-authored-by: Ville Brofeldt <[email protected]>
Co-authored-by: Hugh A. Miles II <[email protected]>
Co-authored-by: josedev-union <[email protected]>
Co-authored-by: yousoph <[email protected]>
Co-authored-by: Jack Fragassi <[email protected]>
josedev-union pushed a commit to Ortege-xyz/studio that referenced this pull request Jan 22, 2024
cccs-rc pushed a commit to CybercentreCanada/superset that referenced this pull request Mar 6, 2024
@mistercrunch mistercrunch added 🍒 3.0.2 Cherry-picked to 3.0.2 🍒 3.0.3 Cherry-picked to 3.0.3 🍒 3.0.4 Cherry-picked to 3.0.4 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 3.1.0 First shipped in 3.1.0 labels Mar 13, 2024
vinothkumar66 pushed a commit to vinothkumar66/superset that referenced this pull request Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/M v3.0 Label added by the release manager to track PRs to be included in the 3.0 branch 🍒 3.0.2 Cherry-picked to 3.0.2 🍒 3.0.3 Cherry-picked to 3.0.3 🍒 3.0.4 Cherry-picked to 3.0.4 🚢 3.1.0 First shipped in 3.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants