Skip to content

Improve homepage UI#164

Merged
zhouzhuojie merged 1 commit into
masterfrom
zz/add-tables-ui
Sep 17, 2018
Merged

Improve homepage UI#164
zhouzhuojie merged 1 commit into
masterfrom
zz/add-tables-ui

Conversation

@zhouzhuojie
Copy link
Copy Markdown
Collaborator

@zhouzhuojie zhouzhuojie commented Sep 14, 2018

image
image

michberr
michberr approved these changes Sep 17, 2018
View reviewed changes
Copy link
Copy Markdown

@michberr michberr left a comment

Choose a reason for hiding this comment

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

Code looks fine, what was the feedback you were responding to?

@zhouzhuojie
Copy link
Copy Markdown
Collaborator Author

zhouzhuojie commented Sep 17, 2018

Sorting flags on updatedAt, and possibly updatedBy

@zhouzhuojie
Copy link
Copy Markdown
Collaborator Author

zhouzhuojie commented Sep 17, 2018

Also

@zhouzhuojie zhouzhuojie merged commit 50a5fe3 into master Sep 17, 2018
@zhouzhuojie zhouzhuojie deleted the zz/add-tables-ui branch September 17, 2018 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rstratton rstratton Awaiting requested review from rstratton

@renjie-checkr renjie-checkr Awaiting requested review from renjie-checkr

@mjramosb mjramosb Awaiting requested review from mjramosb

2 more reviewers

@ziru ziru ziru approved these changes

@michberr michberr michberr approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@zhouzhuojie @ziru @michberr