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

Use resources instead of resource for file controller actions #15754

Merged
merged 1 commit into from Mar 14, 2024

Conversation

hellcp-work
Copy link
Contributor

Changes all the actions to use resources routes, so that it aligns more with how it's used. In practice that means that in update action, there is a file name parameter. Depends on #15753

@github-actions github-actions bot added the Frontend Things related to the OBS RoR app label Mar 7, 2024
@hellcp-work hellcp-work force-pushed the files-controller-p2 branch 2 times, most recently from 20f9e50 to c5a6c84 Compare March 8, 2024 08:03
Copy link

codecov bot commented Mar 8, 2024

Codecov Report

Attention: Patch coverage is 85.71429% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 86.02%. Comparing base (a6f0447) to head (4fa21b2).
Report is 40 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #15754   +/-   ##
=======================================
  Coverage   86.02%   86.02%           
=======================================
  Files         793      793           
  Lines       26011    26015    +4     
=======================================
+ Hits        22375    22379    +4     
  Misses       3636     3636           

@hellcp-work hellcp-work marked this pull request as ready for review March 8, 2024 14:33
@hellcp-work hellcp-work merged commit 8f3d7a5 into openSUSE:master Mar 14, 2024
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend Things related to the OBS RoR app
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants