Skip to content
This repository has been archived by the owner on Jun 11, 2023. It is now read-only.

Add DatabaseIndex #393

Merged
merged 1 commit into from
Mar 9, 2018
Merged

Add DatabaseIndex #393

merged 1 commit into from
Mar 9, 2018

Conversation

renatomassaro
Copy link
Member

@renatomassaro renatomassaro commented Mar 9, 2018 •
edited by umamaistempo
Loading

Incidental

  • Changed return type of build_file at the default ProcessView renderer for easier integration with the client
  • Added bank_accounts information on the AccountIndex

This change is Reviewable

@renatomassaro renatomassaro self-assigned this Mar 9, 2018
@renatomassaro
Copy link
Member Author

Reviewed 9 of 9 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

@sourcelevel-bot
Copy link

Ebert has finished reviewing this Pull Request and has found:

  • 1 fixed issue! 🎉

But beware that this branch is 2 commits behind the HackerExperience:master branch, and a review of an up to date branch would produce more accurate results.

You can see more details about this review at https://ebertapp.io/github/HackerExperience/Helix/pulls/393.

@renatomassaro renatomassaro merged commit 8144f2c into HackerExperience:master Mar 9, 2018
@renatomassaro renatomassaro deleted the client-int branch March 9, 2018 03:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@renatomassaro renatomassaro

Labels
Kind:Enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@renatomassaro