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 routes, tests, views for CSV tag and node_tag stats downloads #5413

Closed
wants to merge 1 commit into from

Conversation

jywarren
Copy link
Member

@jywarren jywarren commented Apr 8, 2019

Relates to #5412, providing routes, tests, and views. I tried to open this as a single first-timers-only issue but the issue at #5412 resulted in only one file's changes, so this is the rest.

@plotsbot
Copy link
Collaborator

plotsbot commented Apr 8, 2019

1 Warning
⚠️ This pull request cannot be merged yet due to merge conflicts. Please resolve them – probably by rebasing – and ask for help (in the comments, or in the chatroom if you get into trouble!.
1 Message
📖 @jywarren Thank you for your pull request! I’m here to help with some tips and recommendations. Please take a look at the list provided and help us review and accept your contribution! And don’t be discouraged if you see errors – we’re here to help.

Generated by 🚫 Danger

@jywarren jywarren closed this Apr 24, 2019
@cesswairimu cesswairimu deleted the first-timers-tag-csv branch May 4, 2021 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants