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.

Sign up for GitHub

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

Jump to bottom

web/satellite: theme and style updates #6860

Merged
merged 7 commits into from Mar 14, 2024
Merged

web/satellite: theme and style updates #6860

merged 7 commits into from Mar 14, 2024

Conversation

boshevski
Copy link
Member

@boshevski boshevski commented Mar 13, 2024 •
edited by andriikotko

Updating color theme and styles, plus few copy updates.

Code Review Checklist (to be filled out by reviewer)

  • NEW: Are there any Satellite database migrations? Are they forwards and backwards compatible?
  • Does the PR describe what changes are being made?
  • Does the PR describe why the changes are being made?
  • Does the code follow our style guide?
  • Does the code follow our testing guide?
  • Is the PR appropriately sized? (If it could be broken into smaller PRs it should be)
  • Does the new code have enough tests? (every PR should have tests or justification otherwise. Bug-fix PRs especially)
  • Does the new code have enough documentation that answers "how do I use it?" and "what does it do?"? (both source documentation and higher level, diagrams?)
  • Does any documentation need updating?
  • Do the database access patterns make sense?

@boshevski
web/satellite: theme and style updates
7ceeb6f 
Updating color theme and styles, plus few copy updates.
@cla-bot cla-bot bot added the cla-signed label Mar 13, 2024
@boshevski boshevski enabled auto-merge (squash) March 13, 2024 11:05
@boshevski boshevski merged commit b3dd07b into main Mar 14, 2024
2 checks passed
@boshevski boshevski deleted the styling-updates branch March 14, 2024 09:54
andriikotko pushed a commit that referenced this pull request Mar 15, 2024
@boshevski @andriikotko
web/satellite: theme and style updates (#6860)
8721ffd 
Updating color theme and styles, plus few copy updates.
andriikotko added a commit that referenced this pull request Mar 15, 2024
@andriikotko @dlamarmorgan
@boshevski @VitaliiShpital @mobyvb
cherry-pick fix storjscan migration (#6866)
53b57d9 
* satellite/satellitedb/migrate: fix migration of storjscan primary key

Change-Id: Ifc27f01267954525391d2a2dea1d9f6a8802d4b2

* web/satellite: theme and style updates (#6860)

Updating color theme and styles, plus few copy updates.

* web/satellite: fixed NaN issue for edit project limit dialog

Added check for undefined value before setting active value dimension.

Issue:
#6848

Change-Id: I989c647e431ba8ace699be7d5969b617ef1c9034

* web/satellite: trigger upgrade flow on add CC or tokens

Updated logic to trigger upgrade account flow on add CC or tokens clicks.

Change-Id: I95d6661248091c541d0c7b33af4d62a957b9f658

* web/satellite: Fix project limit UI bug

On the "all projects" dashboard, projects that a user was invited to,
but did not own, were included in the "total projects" count when
determining whether to allow the user to create a project, or prompt
them to upgrade or request a limit increase.

This change fixes the problem by only counting a user's owned projects
when deciding whether to allow the user to make a "create project"
request.

#6864

Change-Id: I6c86f659d400079e0bafdfa49c4a7f052970b04f

---------

Co-authored-by: dlamarmorgan <damein@storj.io>
Co-authored-by: Tome Boshevski <tome@storj.io>
Co-authored-by: Vitalii <takur12345@gmail.com>
Co-authored-by: Moby von Briesen <mobyvb@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andriikotko andriikotko andriikotko approved these changes

@wilfred-asomanii wilfred-asomanii wilfred-asomanii approved these changes

@jewharton jewharton Awaiting requested review from jewharton jewharton is a code owner

Assignees
No one assigned
Labels
cla-signed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@boshevski @andriikotko @wilfred-asomanii