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

Chat/1.2.2 - Metrics, Hackathons, Notifications. #6108

Merged
merged 71 commits into from
Mar 11, 2020
Merged

Chat/1.2.2 - Metrics, Hackathons, Notifications. #6108

merged 71 commits into from
Mar 11, 2020

Conversation

androolloyd
Copy link
Contributor

@androolloyd androolloyd commented Mar 2, 2020

Description

Chat/1.2.2 - Metrics and Hackathon updates

Refers/Fixes
  • Containerized Chat Added to Docker-compose
  • Chat Data fixture to create an oauth2 application for the chat application
  • Chat metrics added to stats
  • Email Suppression Settings respected
  • Chat Access Token added to context.py
    • notification settings long poll 30s
  • Hackathon Setup Script
    • run as a cron job, will query active Hackathons and ensure that any of the channels needed are created and will add in all the users who are apart of that Hackathon to the sponsor channels as well as the default channels.

Production env variable required

CHAT_SERVER_URL, can be identical to the CHAT_URL given the current setup, the difference is to allow for a different URL should it be needed to connect to the mattermost API from server server, as is the case in the docker network, where it connects via named servers, web, db, chat etc

@androolloyd androolloyd requested a review from owocki March 2, 2020 16:31
@codecov
Copy link

codecov bot commented Mar 2, 2020

Codecov Report

Merging #6108 into stable will decrease coverage by 0.08%.
The diff coverage is 15.44%.

Impacted file tree graph

@@            Coverage Diff             @@
##           stable    #6108      +/-   ##
==========================================
- Coverage   28.51%   28.42%   -0.09%     
==========================================
  Files         278      278              
  Lines       24857    25000     +143     
  Branches     3634     3655      +21     
==========================================
+ Hits         7087     7107      +20     
- Misses      17494    17611     +117     
- Partials      276      282       +6
Impacted Files Coverage Δ
app/app/utils.py 22.18% <0%> (-0.52%) ⬇️
app/marketing/stats.py 0% <0%> (ø) ⬆️
...d/management/commands/post_leaderboard_to_slack.py 0% <0%> (ø) ⬆️
.../management/commands/setup_hackathon_event_chat.py 0% <0%> (ø)
...management/commands/sync_event_bounties_to_chat.py 0% <0%> (ø) ⬆️
app/chat/management/commands/sync_users_to_chat.py 0% <0%> (ø) ⬆️
app/app/settings.py 80.78% <100%> (+0.12%) ⬆️
app/dashboard/models.py 51.03% <100%> (+0.07%) ⬆️
app/chat/tasks.py 15.74% <14.18%> (-7.6%) ⬇️
app/dashboard/views.py 11.78% <2.43%> (-0.06%) ⬇️
... and 7 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 f3d830c...222a389. Read the comment docs.

@androolloyd androolloyd changed the title Chat/1.2.2 - Metrics, Hackathons, Notifications. Chat/1.2.2 - Metrics, Hackathons, Notifications. - Not Ready Mar 4, 2020
…om the chat fixture to the chatconfig/config.json override
…ompose network, fixture setup added to docker-command.bash
…allow for the front end user sessions to query the chat store
…oad requirement for updating the chat user store
@danlipert danlipert merged commit 464e55d into stable Mar 11, 2020
@thelostone-mc thelostone-mc deleted the chat/1.2.2 branch June 27, 2020 00:47
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