Skip to content

Fix create/update push target routes - #7461

Merged
abnegate merged 4 commits into
1.5.xfrom
fix-create-push-target
Jan 19, 2024
Merged

Fix create/update push target routes#7461
abnegate merged 4 commits into
1.5.xfrom
fix-create-push-target

Conversation

@abnegate

@abnegateabnegate commented Jan 17, 2024

Copy link
Copy Markdown
Member

What does this PR do?

Fix account create push target

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Screenshots may also be helpful.)

Related PRs and Issues

  • (Related PR or issue)

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

stnguyen90
stnguyen90 reviewed Jan 18, 2024
View reviewed changes
Comment threadapp/controllers/api/account.php
Comment threadapp/controllers/api/account.php Outdated
abnegateand others added 3 commits January 18, 2024 20:55
@abnegateabnegate changed the title Remove docs label, fix throw on no providerFix create/update push target routesJan 19, 2024
@abnegate
abnegate merged commit 38d0185 into 1.5.xJan 19, 2024
@abnegate
abnegate deleted the fix-create-push-target branch January 19, 2024 05:49
This was referenced Feb 1, 2024
Closed
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@stnguyen90stnguyen90stnguyen90 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@abnegate@stnguyen90