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

removed unused xql files #411

Merged
merged 2 commits into from
Sep 9, 2024
Merged

removed unused xql files #411

merged 2 commits into from
Sep 9, 2024

Conversation

daniel-jettka
Copy link
Contributor

removed:

  • getAnnotationParticipants.xql
  • loginToTextGrid.xql
  • checkTextGridLogin.xql
  • getAnnotationsInRendering.xql

combines issues #366 #322 and #324

@daniel-jettka daniel-jettka linked an issue Aug 27, 2024 that may be closed by this pull request
1 task
@krHERO krHERO requested a review from bwbohl September 5, 2024 08:06
@bwbohl bwbohl added this to the 1.0.0 milestone Sep 5, 2024
@peterstadler peterstadler linked an issue Sep 6, 2024 that may be closed by this pull request
bwbohl
bwbohl previously approved these changes Sep 7, 2024
@daniel-jettka daniel-jettka dismissed bwbohl’s stale review September 7, 2024 08:36

The merge-base changed after approval.

@bwbohl bwbohl self-requested a review September 7, 2024 08:36
bwbohl
bwbohl previously approved these changes Sep 7, 2024
@bwbohl
Copy link
Member

bwbohl commented Sep 7, 2024

The branch had some merge conflicts that had to be resolved locally, so I rebased it on develop and pushed it. Unfortunately this means we need someone else to review: @Edirom/edirom-online-developer please feel free ;-)

@daniel-jettka daniel-jettka dismissed bwbohl’s stale review September 7, 2024 09:55

The merge-base changed after approval.

@peterstadler
Copy link
Member

Sorry, but the current PR looks rather polluted now with 93 commits and 50 changed files. My recommendation would be to clean this mess up and force push the branch.

@bwbohl
Copy link
Member

bwbohl commented Sep 9, 2024

sorry for that, obviously something went wrong here…

@bwbohl bwbohl force-pushed the ftr/366-remove-unused-xql branch from d46e2ea to c71f856 Compare September 9, 2024 09:23
@bwbohl
Copy link
Member

bwbohl commented Sep 9, 2024

ok fixed it

@bwbohl bwbohl merged commit d44dfd0 into develop Sep 9, 2024
1 check passed
@bwbohl bwbohl deleted the ftr/366-remove-unused-xql branch September 9, 2024 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Remove unused xql files
3 participants