Skip to content
This repository was archived by the owner on May 25, 2021. It is now read-only.

Fixing issue where two : were used for prefixes of local files, and r… #136

Merged
james-martin-jd merged 1 commit into from
Nov 1, 2019
Merged

Fixing issue where two : were used for prefixes of local files, and r… #136

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions src/demo/App.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -96,7 +96,8 @@ const App = () => {
// Discover the inbox url from the resource, using ldp:inbox predicate
const inboxUrl = await discoverInbox(userWebID);
// The actor in this case is the current application, so we can use the current URL
const actor = window.location.href;
// Removing actor temporarily until we figure out how to link applications
// const actor = window.location.href;

if (!inboxUrl) {
throw new Error('Inbox not found');
Expand All @@ -106,7 +107,7 @@ const App = () => {
{
title: 'Notification Example',
summary: 'This is a basic solid notification example.',
actor
actor: 'https://solidsdk.inrupt.net/profile/card#me'
},
inboxUrl,
NotificationTypes.ANNOUNCE
Expand Down
2 changes: 1 addition & 1 deletion src/lib/classes/notification.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -246,7 +246,7 @@ export class Notification {

const { '@context': context, shape } = this.schema[name];
// Add local filename as a prefix for cleaner ttl generation
const fullContext = { ...context, ':': filePath };
const fullContext = { ...context, '': filePath };

/**
* N3 is an implementation of the RDF.js low-level specification that lets you handle RDF in JavaScript easily.
Expand Down