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

Allow to set agenda item tags with agenda_tag_ids #1832

Merged
merged 3 commits into from
Jul 31, 2023

Conversation

reiterl
Copy link
Member

@reiterl reiterl commented Jul 26, 2023

Remove topic/tag_ids because it is not used in the client.
Update topic.create.

@reiterl reiterl added the enhancement General enhancement which is neither bug nor feature label Jul 26, 2023
@reiterl reiterl added this to the 4.1 milestone Jul 26, 2023
@reiterl reiterl self-assigned this Jul 26, 2023
@reiterl reiterl linked an issue Jul 26, 2023 that may be closed by this pull request
tests/system/action/topic/test_create.py Show resolved Hide resolved
tests/system/action/topic/test_create.py Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

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

Same for these tests

Copy link
Member Author

@reiterl reiterl Jul 29, 2023

Choose a reason for hiding this comment

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

The topic.update action doesn't have set tag_ids anymore and no agenda_tag_ids. So these tests are not needed anymore.

Copy link
Contributor

Choose a reason for hiding this comment

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

Then please transfer these tests to the agenda item update tests. They do not only test the action, but also the relation and action handling and should therefore not be removed.

Copy link
Member Author

Choose a reason for hiding this comment

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

Okay, I have moved the removed tag_ids update to agenda_item.

@reiterl reiterl merged commit 332fcad into OpenSlides:main Jul 31, 2023
4 checks passed
@reiterl reiterl deleted the update-topic-create branch July 31, 2023 08:01
peb-adr added a commit to peb-adr/openslides-backend that referenced this pull request Aug 22, 2023
* commit '6195994174e549f3c7f16ae648027e557eda5fcc': (99 commits)
  Add check to forbid self vote delegation (OpenSlides#1875)
  Fix agenda item update (OpenSlides#1861)
  Fix delete user with poll candidate (OpenSlides#1872)
  Fix of a problem with calculated user/committee_ids (OpenSlides#1859)
  Add default to meeting/users_pdf_wlan_encryption (OpenSlides#1863)
  Add meeting_user_id to user.create action result (OpenSlides#1866)
  change saml_private_key type to text (OpenSlides#1860)
  Move additional model code to mixins (OpenSlides#1808)
  Add saml_id to the account import actions (OpenSlides#1821)
  Fix relation handling with calculated fields handler (OpenSlides#1839)
  Extend project automation
  Update GitHub actions (OpenSlides#1822)
  Fix for a key error problem in speaker create (OpenSlides#1838)
  Allow to set agenda item tags with agenda_tag_ids (OpenSlides#1832)
  Add two options to workflow import (OpenSlides#1833)
  LOS restrict point of order submission to open lists (OpenSlides#1830)
  Revert "Add an unique check for motion_category.prefix. (OpenSlides#1626)" (OpenSlides#1828)
  Update gunicorn & datastore (OpenSlides#1825)
  Rename database env vars & update dependencies (OpenSlides#1800)
  Add tag_ids to topic.create (OpenSlides#1819)
  ...
peb-adr added a commit that referenced this pull request Sep 6, 2023
* commit '9ca701b83a47b2b35cba4a6a85393d45e98807da': (96 commits)
  Update linear sort error messages (#1891)
  Use automation access token in repository_dispatch (#1890)
  fix key error in get_user_scope (#1887)
  Fix reachability of actions in dev mode (#1818)
  Catch internal auth password decoding error (#1817)
  Add check to forbid self vote delegation (#1875)
  Fix agenda item update (#1861)
  Fix delete user with poll candidate (#1872)
  Fix of a problem with calculated user/committee_ids (#1859)
  Add default to meeting/users_pdf_wlan_encryption (#1863)
  Add meeting_user_id to user.create action result (#1866)
  change saml_private_key type to text (#1860)
  Move additional model code to mixins (#1808)
  Add saml_id to the account import actions (#1821)
  Fix relation handling with calculated fields handler (#1839)
  Extend project automation
  Update GitHub actions (#1822)
  Fix for a key error problem in speaker create (#1838)
  Allow to set agenda item tags with agenda_tag_ids (#1832)
  Add two options to workflow import (#1833)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement General enhancement which is neither bug nor feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change topic.create payload
2 participants