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

Add identical motions #846

Merged
merged 2 commits into from
Feb 19, 2024
Merged

Conversation

jsangmeister
Copy link
Contributor

@jsangmeister jsangmeister added this to the 4.2 milestone Feb 14, 2024
@jsangmeister jsangmeister self-assigned this Feb 14, 2024
@jsangmeister jsangmeister added the waiting Waiting for some other PR/feature; more details in comments label Feb 14, 2024
@ostcar
Copy link
Member

ostcar commented Feb 14, 2024

I don't get this PR. The changes in the meta-repo seem not to be in main. Should this be a feature branch? If not, you should first merge the changes in the meta repo

@ostcar
Copy link
Member

ostcar commented Feb 14, 2024

If you don't want to merge this yet, then please change this in a Draft PR

@jsangmeister jsangmeister merged commit 1869e9b into OpenSlides:main Feb 19, 2024
2 checks passed
@jsangmeister jsangmeister deleted the identical-motions branch February 19, 2024 09:48
@@ -5135,6 +5135,15 @@ func (r *Fetch) Motion_ID(motionID int) *ValueInt {
return &ValueInt{fetch: r, key: key}
}

func (r *Fetch) Motion_IDenticalMotionIDs(motionID int) *ValueIntSlice {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ostcar Do you somewhere to a hard replace on "id" to write it uppercase? :D this function name seems suspicious....

Copy link
Member

Choose a reason for hiding this comment

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

It seems this way :)

Here is the place:

name = strings.ReplaceAll(name, "Id", "ID")

Maybe it needs a better heuristic.

peb-adr added a commit that referenced this pull request Mar 12, 2024
* commit '441c4d9b6e7aff5f82baa8bbbdde05240e6436f8': (27 commits)
  Update meta repository (#885)
  Update meta (#877)
  Automatically create PRs for commits on staging branches (#865)
  Change restricter for motion_change_recommendation (#857)
  Bump github.com/klauspost/compress from 1.17.6 to 1.17.7 (#856)
  Remove update-generated-files.yml (#854)
  Update meta repository (#853)
  Bump github.com/gomodule/redigo from 1.8.9 to 1.9.1 (#852)
  new settings for one participant multiple times in the LoS (#851)
  Add identical motions (#846)
  Enhance motion fields (#844)
  Update go version (#850)
  Add motion restriction mode for `is_internal` (#847)
  Unify name of the openslides-meta submodule (#843)
  Add new restrictions for LoS extension (#807) (#810)
  Update meta repository (#841)
  Bump golang.org/x/sys from 0.16.0 to 0.17.0 (#839)
  Bump github.com/klauspost/compress from 1.17.5 to 1.17.6 (#836)
  Use models.yml from meta repo (#834)
  Bump github.com/jackc/pgx/v5 from 5.5.2 to 5.5.3 (#835)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature waiting Waiting for some other PR/feature; more details in comments
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants