Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: updating readme and display version for rebranding #2550

Merged
merged 14 commits into from
Aug 25, 2022

Conversation

ashin-czi
Copy link
Contributor

#309

Reviewers

Functional: @seve

Readability: @seve @karenliang0


Changes

  • add
  • remove
  • modify
    Updating Readme and init.py

Copy link
Contributor

@ambrosejcarr ambrosejcarr left a comment

Choose a reason for hiding this comment

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

Approved pending two suggested changes. Thanks.

README.md Outdated Show resolved Hide resolved
server/__init__.py Outdated Show resolved Hide resolved
Copy link
Contributor

@ambrosejcarr ambrosejcarr left a comment

Choose a reason for hiding this comment

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

Currently the browser tab shows as:

image

For CZ CELLxGENE Annotate, I'd like the browser icon to read:

[NEW pink cellxgene logo] CELLxGENE | Annotate

This will establish a visual distinction from Discover applications but keep the look & feel consistent across our applications.

README.md Outdated Show resolved Hide resolved
- CELLxGENE -> CELLxGENE Annotate
- reverting __init__.py display_version back to "cellxgene"
@ashin-czi
Copy link
Contributor Author

Made the changes per comments above

Copy link
Contributor

@ambrosejcarr ambrosejcarr left a comment

Choose a reason for hiding this comment

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

LGTM.

@ashin-czi ashin-czi enabled auto-merge (squash) August 9, 2022 19:48
client/index.html Show resolved Hide resolved
client/index_template.html Show resolved Hide resolved
client/src/components/app.js Show resolved Hide resolved
@ashin-czi ashin-czi requested a review from seve August 9, 2022 21:48
@codecov
Copy link

codecov bot commented Aug 25, 2022

Codecov Report

Merging #2550 (598a298) into main (ff5da77) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2550   +/-   ##
=======================================
  Coverage   71.60%   71.60%           
=======================================
  Files          96       96           
  Lines        6652     6652           
  Branches      770      770           
=======================================
  Hits         4763     4763           
  Misses       1813     1813           
  Partials       76       76           
Flag Coverage Δ
frontend 71.60% <ø> (ø)
javascript 71.60% <ø> (ø)
smokeTest ?
unitTest 71.60% <ø> (ø)

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

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ashin-czi ashin-czi merged commit c0f4ad6 into main Aug 25, 2022
@ashin-czi ashin-czi deleted the ashin-czi/docs-rebrand-cellxgene branch August 25, 2022 15:41
@ashin-czi
Copy link
Contributor Author

Alec was able to figure out the unit tests issue so this PR is now merged!

@ashin-czi
Copy link
Contributor Author

Ready for staging

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.

4 participants