Details
-
Type:
Improvement
-
Status:
Resolved
-
Priority:
Major
-
Resolution: Obsolete
-
Affects Version/s: Current Version
-
Fix Version/s: Far Future
-
Component/s: OIOIOI
-
Labels:
Description
And this everything just to show scores in submissions diff view...
Current design doesn't cooperate well with cross-users submission privileges. Sometimes (namely in cheaters app, source publicizing) we want to show source code (and score in the latter) to non-owner and non-admin, but reusing existing views (especially for source) breaks, when other part of the code want's to show there some useful info.
What we have thought so far:
* It would be really great, to not have to worry in templates whether we should add some diff to check is value is visible. We can achieve that by creating wrapper over Submission objects, which sets to None inaccessible attributes.
* (already forgot)
Current design doesn't cooperate well with cross-users submission privileges. Sometimes (namely in cheaters app, source publicizing) we want to show source code (and score in the latter) to non-owner and non-admin, but reusing existing views (especially for source) breaks, when other part of the code want's to show there some useful info.
What we have thought so far:
* It would be really great, to not have to worry in templates whether we should add some diff to check is value is visible. We can achieve that by creating wrapper over Submission objects, which sets to None inaccessible attributes.
* (already forgot)
Activity
- All
- Comments
- History
- Activity
- Transitions
- Commits
Szymon Acedański
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Status | New [ 10000 ] | Open [ 1 ] |
Szymon Acedański
made changes -
| Fix Version/s | Near Future [ 11001 ] |
Maciej Dębski
made changes -
| Summary | Rework Submissions' Access Control To Enable More Robust And Flexible Determining What Should Be Show And What Not™ | Rework Submissions' Access Control To Enable More Robust And Flexible Determining What Should Be Shown And What Not™ |
Szymon Acedański
made changes -
| Affects Version/s | Current Version [ 11000 ] | |
| Affects Version/s | Near Future [ 11001 ] |
Szymon Acedański
made changes -
| Fix Version/s | Far Future [ 11002 ] | |
| Fix Version/s | Near Future [ 11001 ] |
Szymon Acedański
made changes -
| Status | Open [ 1 ] | Resolved [ 5 ] |
| Assignee | Szymon Acedański [ accek ] | |
| Resolution | Obsolete [ 7 ] |
