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

Website-redesign: Implement about page from hero to quote section #5853

Merged
merged 23 commits into from
Sep 14, 2020

Conversation

michellewong793
Copy link
Contributor

@michellewong793 michellewong793 commented Sep 3, 2020

This starts implementing the About page, completes desktop version up to the quote section.

screencapture-localhost-8000-about-2020-09-09-11_01_58.pdf

@michellewong793 michellewong793 requested review from a team as code owners September 3, 2020 20:27
michellewong793 and others added 16 commits September 8, 2020 11:33
* Moves fee_payer_dec to fee_transfer when in a block to be more
faithful to protocol's understanding of operations; mempool still
returns fee_payer_dec from user_command as an approximation
* Fixes JOIN clause on SQL queries utilizing the join table for blocks
and transactions to properly query for them.

`rosetta-cli check:data` now works against everything our test-agent
throws at it (which at the moment are payments and delegations):

```
+--------------------+--------------------------------+--------+
| Response Assertion | All responses are correctly    | PASSED |
|                    | formatted                      |        |
+--------------------+--------------------------------+--------+
| Block Syncing      | Blocks are connected into a    | PASSED |
|                    | single canonical chain         |        |
+--------------------+--------------------------------+--------+
| Balance Tracking   | Account balances did not go    | PASSED |
|                    | negative                       |        |
+--------------------+--------------------------------+--------+
| Reconciliation     | No balance discrepencies were  | PASSED |
|                    | found between computed and     |        |
|                    | live balances                  |        |
+--------------------+--------------------------------+--------+
```
@michellewong793 michellewong793 force-pushed the website-redesign/about-page branch from 577b36a to e611271 Compare September 8, 2020 20:26
@michellewong793 michellewong793 changed the title About Page Website-redesign: About page Sep 8, 2020
@michellewong793 michellewong793 changed the title Website-redesign: About page Website-redesign: Implement About page Sep 8, 2020
@michellewong793 michellewong793 changed the title Website-redesign: Implement About page Website-redesign: About page Quote component up to hero section Sep 9, 2020
@michellewong793 michellewong793 changed the title Website-redesign: About page Quote component up to hero section Website-redesign: Implement about page from hero to quote section Sep 9, 2020
Copy link
Contributor

@figitaki figitaki left a comment

Choose a reason for hiding this comment

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

Looks okay for now, as long as we break out the reusable components and handle the TODOs

@michellewong793 michellewong793 added ci-build-me Add this label to trigger a circle+buildkite build for this branch ready-to-merge Adding this label will trigger mergify and trigger CI to run and merge the PR labels Sep 14, 2020
@netlify
Copy link

netlify bot commented Sep 14, 2020

Preview:

Built with commit 4bd6f29

https://deploy-preview-5853--o1website2.netlify.app

@mergify mergify bot merged commit f5119d0 into develop Sep 14, 2020
@mergify mergify bot deleted the website-redesign/about-page branch September 14, 2020 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-build-me Add this label to trigger a circle+buildkite build for this branch ready-to-merge Adding this label will trigger mergify and trigger CI to run and merge the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants