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

Hackathon Townsquare - General Cleanup and Project Panel Updates #6439

Merged
merged 12 commits into from
Apr 22, 2020

Conversation

androolloyd
Copy link
Contributor

@androolloyd androolloyd commented Apr 15, 2020

Description
  • Adding Looking for members Filter
  • Project Panel updates
  • message team members from project panel
Refers/Fixes

Issue with adding members to channel, users who have left the chat experience halt the mass adding of hackers.

Testing

@codecov
Copy link

codecov bot commented Apr 15, 2020

Codecov Report

Merging #6439 into master will increase coverage by 0.67%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6439      +/-   ##
==========================================
+ Coverage   27.18%   27.85%   +0.67%     
==========================================
  Files         289      289              
  Lines       26463    29034    +2571     
  Branches     3922     4840     +918     
==========================================
+ Hits         7194     8088     +894     
- Misses      18999    20559    +1560     
- Partials      270      387     +117     
Impacted Files Coverage Δ
app/chat/tasks.py 14.01% <0.00%> (-0.14%) ⬇️
app/dashboard/views.py 13.10% <0.00%> (+1.85%) ⬆️
app/app/urls.py 83.33% <0.00%> (-3.11%) ⬇️
app/grants/views.py 15.00% <0.00%> (-0.97%) ⬇️
app/dashboard/tasks.py 37.70% <0.00%> (-0.63%) ⬇️
app/kudos/views.py 14.05% <0.00%> (-0.41%) ⬇️
app/quests/views.py 16.01% <0.00%> (-0.29%) ⬇️
app/marketing/views.py 11.50% <0.00%> (-0.11%) ⬇️
app/dashboard/helpers.py 14.06% <0.00%> (-0.10%) ⬇️
app/dashboard/context/hackathon.py 0.00% <0.00%> (ø)
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e234f5f...b373347. Read the comment docs.

Copy link
Contributor

@octavioamu octavioamu left a comment

Choose a reason for hiding this comment

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

Left some questions im not sure If Im missing something, but ping me if anything.

<script>
$('body').bootstrapTooltip({
selector: '[data-toggle="tooltip"]'
});
const csrftoken = jQuery("[name=csrfmiddlewaretoken]").val();
document.keywords = {{ keywords | safe }};
document.is_registered = {{ is_registered | safe }};
document.is_registered = JSON.parse(document.getElementById('hackathon-registration').textContent);
Copy link
Contributor

Choose a reason for hiding this comment

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

where is this element with id hackathon-registration ?

app/dashboard/views.py Outdated Show resolved Hide resolved
app/dashboard/templates/dashboard/index-vue.html Outdated Show resolved Hide resolved
@octavioamu octavioamu merged commit 3a9fda3 into master Apr 22, 2020
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.

4 participants