-
Notifications
You must be signed in to change notification settings - Fork 2
Create a post on Bluesky when a dataset is made public #544
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- Add hashtags (different hash tags when posting to test account) - Added checkbox in ConfirmView to "Post to test account"
- Added option in Bluesky settings to save Panorama log file.
…y admin console - Added BlueskySettings and BlueskySettingsManager classes - Added option to auto-post to Bluesky when a dataset is made public by submitter. Confirmation message includes the Bluesky post URL.
- Added buttons for "Post to primary account" and "Post to test account". - Save the AT uri only if posting to primary account. - Announcement text is configurable through the settings page.
- Report account and endpoint in exception message.
…kySettingsManager. Fix typos.
| sql.append(parentColumn).append(" IN (SELECT EntityId FROM ") | ||
| .append(CoreSchema.getInstance().getTableInfoContainers(), "c").append(")") | ||
| .append(" AND (") | ||
| .append(documentNameColumn).append(" LIKE '").append(PanoramaPublicLogoManager.LOGO_FILE_PREFIX).append("%' )"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This constant is unlikely to change to a dangerous value, but safer to use SQLFragment.appendStringLiteral()
| catch (BlueskyException e) | ||
| { | ||
| _exception = e; | ||
| LOG.error(e); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this will just get you the toString(). Maybe that's enough here, but you might want the two-argument version which will get you the full stack trace
…PublicModule.java - CR feedback
Rationale
We would like to post a message to the Bluesky social media platform after a dataset is made public on Panorama Public.
Changes