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

Add Transfers Report #599

Merged
merged 8 commits into from
Mar 15, 2021
Merged

Add Transfers Report #599

merged 8 commits into from
Mar 15, 2021

Conversation

markets
Copy link
Collaborator

@markets markets commented Mar 13, 2021

First part of #596 (partially extracted from #259)

EXTRA

  • Open Tags section for all users. Now that we have a good ✔️ responsive page (since Set col to a different value #547), this is very useful to explore all the content inside an organization

@markets markets mentioned this pull request Mar 13, 2021
3 tasks
@markets markets added the wip label Mar 13, 2021
@markets markets removed the wip label Mar 13, 2021
@markets
Copy link
Collaborator Author

markets commented Mar 13, 2021

@sseerrggii this is ready to test :bowtie:

@sseerrggii
Copy link
Contributor

Thanks @markets Good first step for #596 Also I agree to open Tags section for all users 😉

I tested on staging and I got this error

Screenshot_2021-03-15 Banco de Tiempo Local

it occurred at https://staging.timeoverflow.org/statistics/statistics_all_transfers and at https://staging.timeoverflow.org/report/transfer_list

Info from Rollbar:

Screenshot_2021-03-15 Rollbar

@markets
Copy link
Collaborator Author

markets commented Mar 15, 2021

Thanks @sseerrggii, I'll take a look later, I see what's happening.

@markets
Copy link
Collaborator Author

markets commented Mar 15, 2021

@sseerrggii I just rebased develop and added a fix for that nil pointer exception

@sseerrggii
Copy link
Contributor

Tested! 🟢

@sseerrggii sseerrggii merged commit 92c8cf9 into develop Mar 15, 2021
@markets markets deleted the more_reports branch March 15, 2021 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants