Skip to content

Conversation

@flacial
Copy link
Member

@flacial flacial commented Oct 24, 2022

Related to #1990

The actual changes were accidentally included in this PR. This PR will serve as a future reference.

Description

Merging this PR will create the page that'll be used to manage flagged & removed exercises. In a subsequent PR, it'll filter removed exercises from flagged and add the filtering input box.

Testing

  1. Add exercises from /curriculum/js0/mentor/addExercise
  2. Go to the page under /admin/lessons/js0/exercises
  3. Make sure remove and unflag actions work as intended (the exercises should refresh upon performing any of the actions)

@vercel
Copy link

vercel bot commented Oct 24, 2022

@flacial is attempting to deploy a commit to the c0d3-prod Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Oct 24, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
c0d3-app ✅ Ready (Inspect) Visit Preview Oct 25, 2022 at 1:50AM (UTC)

@flacial
Copy link
Member Author

flacial commented Oct 24, 2022

The changes were accidentally merged in this PR

@flacial flacial closed this Oct 24, 2022
@flacial flacial reopened this Oct 24, 2022
@codecov
Copy link

codecov bot commented Oct 24, 2022

Codecov Report

Merging #2448 (7740086) into master (3b31d38) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master     #2448   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          185       185           
  Lines         3307      3307           
  Branches       878       878           
=========================================
  Hits          3307      3307           
Impacted Files Coverage Δ
...es/admin/lessons/[lessonSlug]/[pageName]/index.tsx 100.00% <ø> (ø)

@flacial
Copy link
Member Author

flacial commented Oct 24, 2022

It was reopened to serve the purpose of a future reference.

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