Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove raw problem view #1801

Merged
merged 1 commit into from
Oct 16, 2021
Merged

Remove raw problem view #1801

merged 1 commit into from
Oct 16, 2021

Conversation

kiritofeng
Copy link
Member

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Oct 14, 2021

Codecov Report

Merging #1801 (e5307c2) into master (981018a) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1801   +/-   ##
=======================================
  Coverage   46.17%   46.18%           
=======================================
  Files         223      223           
  Lines       12683    12666   -17     
=======================================
- Hits         5857     5850    -7     
+ Misses       6826     6816   -10     
Impacted Files Coverage Δ
dmoj/urls.py 89.74% <ø> (ø)
judge/views/problem.py 24.53% <ø> (-0.56%) ⬇️

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 981018a...e5307c2. Read the comment docs.

@int-y1
Copy link
Contributor

int-y1 commented Oct 14, 2021

Other places in the codebase use problem/raw.html -- make sure that pdf generation does not break

@Ninjaclasher
Copy link
Member

Don't forget about the robots.txt entry.

@kiritofeng kiritofeng marked this pull request as ready for review October 16, 2021 01:12
@quantum5 quantum5 merged commit 0737e52 into master Oct 16, 2021
@quantum5 quantum5 deleted the remove-problem-raw branch October 16, 2021 01:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants