Skip to content

Commit

Permalink
fix: Don't load settings when first initialising Publisher. Fixes iss…
Browse files Browse the repository at this point in the history
…ue when no settings available in set up situation.
  • Loading branch information
andymac4182 committed Apr 28, 2023
1 parent 1acc9b8 commit ceb21e7
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion packages/lib/src/Publisher.ts
Expand Up @@ -101,6 +101,7 @@ export class Publisher {
settings: ConfluenceSettings;
mermaidRenderer: MermaidRenderer;
myAccountId: string | undefined;
settingsLoader: SettingsLoader;

constructor(
adaptor: LoaderAdaptor,
Expand All @@ -109,13 +110,15 @@ export class Publisher {
mermaidRenderer: MermaidRenderer
) {
this.adaptor = adaptor;
this.settings = settingsLoader.load();
this.settingsLoader = settingsLoader;
this.mermaidRenderer = mermaidRenderer;

this.confluenceClient = confluenceClient;
}

async publish(publishFilter?: string) {
this.settings = this.settingsLoader.load();

if (!this.myAccountId) {
const currentUser =
await this.confluenceClient.users.getCurrentUser();
Expand Down

0 comments on commit ceb21e7

Please sign in to comment.