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

Support public tasks #418

Closed
wants to merge 5 commits into from
Closed

Support public tasks #418

wants to merge 5 commits into from

Conversation

hannah-martinez
Copy link
Member

@hannah-martinez hannah-martinez commented Dec 16, 2024

Note that I define a public task as a task available to an authenticated, unauthorized user. Authenticated means they've logged in with a Google account, unauthorized means that an admin user has not yet moved this user to the AuthorizedUsers group in the Django console.

  • Add 40 tasks to a namespace called Public Tasks Test (WIP). Assign all tasks to user "public" at creation.
  • Allow public users access to the task page, workspace page, and inspect task page.
  • Allow public users access to open & pending tasks which are assigned to user "public". Hide these tasks from authorized users.
  • Allow public users access to closed tasks assigned to themselves.
  • When task is submitted, use the task ID instead of get_next_task()
  • Reassign public tasks to submitting username on close. (Still need to think through security implications of this)
  • Add a new tab called Shortcuts to the dashboard page and add the ability to view all tasks assigned to user "public" through this new tab. This is helpful because no google account corresponds to the "public" assignee. We can add other shortcuts in the future if we want.
  • Add contact email to task page for unauthorized user
  • Fix a bug associated with choosing NO to tracking operation IDs when creating a namespace
  • Fix a spellcheck bug for the bot

…d to tasks with assignee 'public'. Hide any open/pending tasks from unauthorized users even if they are assigned to them
@hannah-martinez hannah-martinez self-assigned this Dec 16, 2024
@hannah-martinez hannah-martinez marked this pull request as draft December 16, 2024 20:57
Copy link
Collaborator

@dxenes1 dxenes1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • create task bucket for public and pull from that instead of hard-coded assignee.
  • if a user is not authorized, pull from public and immediately re-assign to their own username. when they close the task, close with their username.
  • (OPTIONAL) add expiration date to tasks to ensure tasks that remain uncompleted return back to queue

@dxenes1
Copy link
Collaborator

dxenes1 commented Feb 5, 2025

Closing to merge with #419

@dxenes1 dxenes1 closed this Feb 5, 2025
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