Skip to content

Conversation

@come-nc
Copy link
Contributor

@come-nc come-nc commented Sep 16, 2025

Summary

It’s rare but exists for some apps not in the appstore.
Also added unit tests for cleanAppId and fixed small issues with it.

Checklist

@come-nc come-nc added this to the Nextcloud 33 milestone Sep 16, 2025
@come-nc come-nc self-assigned this Sep 16, 2025
@come-nc come-nc added bug 3. to review Waiting for reviews labels Sep 16, 2025
@come-nc
Copy link
Contributor Author

come-nc commented Sep 16, 2025

/backport to stable32

@come-nc
Copy link
Contributor Author

come-nc commented Sep 16, 2025

/backport to stable31

@come-nc
Copy link
Contributor Author

come-nc commented Sep 16, 2025

/backport to stable30

@come-nc come-nc marked this pull request as ready for review September 16, 2025 13:11
@come-nc come-nc requested a review from a team as a code owner September 16, 2025 13:11
@come-nc come-nc requested review from yemkareems and removed request for a team September 16, 2025 13:11
Copy link
Collaborator

@Altahrim Altahrim left a comment

Choose a reason for hiding this comment

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

Maybe add a test to validate numbers are kept in middle?

Copy link
Contributor

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

LGTM, thanks a lot! ☺️

It’s rare but exists for some apps not in the appstore.
Also added unit tests for cleanAppId and fixed small issues with it.

Signed-off-by: Côme Chilliet <[email protected]>
@come-nc come-nc force-pushed the fix/allow-hyphen-in-appid branch from b846ea7 to ae4d38e Compare September 16, 2025 15:59
@come-nc come-nc enabled auto-merge September 16, 2025 16:16
@come-nc come-nc merged commit 7e0cda9 into master Sep 16, 2025
220 of 226 checks passed
@come-nc come-nc deleted the fix/allow-hyphen-in-appid branch September 16, 2025 21:46
@backportbot
Copy link

backportbot bot commented Sep 16, 2025

The backport to stable30 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable30
git pull origin stable30

# Create the new backport branch
git checkout -b backport/55142/stable30

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick ae4d38eb

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/55142/stable30

Error: Failed to check for changes with origin/stable30: No changes found in backport branch


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot
Copy link

backportbot bot commented Sep 16, 2025

The backport to stable31 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable31
git pull origin stable31

# Create the new backport branch
git checkout -b backport/55142/stable31

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick ae4d38eb

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/55142/stable31

Error: Failed to push branch backport/55142/stable31: remote: Invalid username or token. Password authentication is not supported for Git operations.
fatal: Authentication failed for 'https://github.com/nextcloud/server.git/'


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@skjnldsv skjnldsv added this to the Nextcloud 32 milestone Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants