Skip to content

Commit

Permalink
Revert "13.17.x patch (#1238)" (#1239)
Browse files Browse the repository at this point in the history
This reverts commit 58f160b.
  • Loading branch information
mjuniper authored Sep 27, 2023
1 parent 58f160b commit 344e5e2
Show file tree
Hide file tree
Showing 4 changed files with 32 additions and 24 deletions.
1 change: 0 additions & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ on:
- beta
# - alpha disabled to allow push to alpha then merge to master w/o incurring a release
- next
- [0-9]+.[0-9]+.x
# Dont run if it's just markdown or doc files
paths-ignore:
- "**.md"
Expand Down
48 changes: 30 additions & 18 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion packages/common/src/sites/HubSite.ts
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ export class HubSite
): IHubSite {
// ensure we have the orgUrlKey
if (!partialSite.orgUrlKey) {
partialSite.orgUrlKey = context.portal?.urlKey;
partialSite.orgUrlKey = context.portal.urlKey;
}
// extend the partial over the defaults
const pojo = { ...DEFAULT_SITE, ...partialSite } as IHubSite;
Expand Down
5 changes: 1 addition & 4 deletions packages/common/test/sites/HubSite.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -52,10 +52,7 @@ describe("HubSite Class:", () => {
describe("static methods:", () => {
it("loads from minimal json", () => {
const createSpy = spyOn(HubSitesModule, "createSite");
const chk = HubSite.fromJson(
{ name: "Test Site" },
unauthdCtxMgr.context
);
const chk = HubSite.fromJson({ name: "Test Site" }, authdCtxMgr.context);

expect(createSpy).not.toHaveBeenCalled();
expect(chk.toJson().name).toEqual("Test Site");
Expand Down

0 comments on commit 344e5e2

Please sign in to comment.