Restrict workplan tab actions

[jadmsaadaot]

https://app.zenhub.com/workspaces/epictrack-63891ea941d309001fa292cf/issues/gh/bcgov/epic.track/1579

Details:

• Use Restrict component to wrap buttons
• Add auth_check for all the services in the backend
• Add work_id in the request body to some bulk operations to be able to use it in auth_check

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ No coverage uploaded for pull request base (develop@bf2fcba). Click here to learn what that means.

Additional details and impacted files

@@            Coverage Diff             @@
##             develop    #1672   +/-   ##
==========================================
  Coverage           ?   63.06%           
==========================================
  Files              ?      248           
  Lines              ?     8235           
  Branches           ?        0           
==========================================
  Hits               ?     5193           
  Misses             ?     3042           
  Partials           ?        0           

Flag	Coverage Δ
epictrack-api	63.06% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jadmsaadaot]

@dinesh-aot @saravanpa-aot should we add a "Delete" role for operations like this. We only have Create and Edit now

[dinesh-aot]

Good Question. I think we should create a role called Delete. @saravanpa-aot thoughts?

[jadmsaadaot]

Added this ticket for the Delete role https://app.zenhub.com/workspaces/epictrack-63891ea941d309001fa292cf/issues/gh/bcgov/epic.track/1679, happy to merge this once approved

[jadmsaadaot]

pylint is complaining that the module services/event.py is too big, exceeds lines of code limit. Fixing that would some refactoring and is really out of scope of this, can we ignore it?

[saravanpa-aot]

pylint is complaining that the module services/event.py is too big, exceeds lines of code limit. Fixing that would some refactoring and is really out of scope of this, can we ignore it?

sure... plz create a ticket for refactoring..put the above screenshot

[jadmsaadaot]

pylint is complaining that the module services/event.py is too big, exceeds lines of code limit. Fixing that would some refactoring and is really out of scope of this, can we ignore it?

sure... plz create a ticket for refactoring..put the above screenshot

added https://app.zenhub.com/workspaces/epictrack-63891ea941d309001fa292cf/issues/gh/bcgov/epic.track/1681