Skip to content

[MASTER] fix: make this example work for public api - #22

Merged
ysavicsh merged 1 commit into
masterfrom
fix-make-it-work
Aug 26, 2026
Merged

ysavicsh merged 1 commit into
masterfrom
fix-make-it-work

Conversation

@ysavicsh

@ysavicsh ysavicsh commented Aug 26, 2026

Copy link
Copy Markdown
Contributor
  • update libraries
  • make the service run for both psd2 and public api

@ysavicsh ysavicsh self-assigned this Aug 26, 2026
@ysavicsh
ysavicsh marked this pull request as ready for review August 26, 2026 12:17
@ysavicsh
ysavicsh requested a review from a team as a code owner August 26, 2026 12:17
Comment thread server/config/config.example.json Outdated
Comment on lines +2 to +3
"CLIENT_ID": "996fc83d-97d7-48f1-9638-744d89874a8c",
"CLIENT_SECRET": "2fd0b7558b43c8b0c078ef2126a79eee336e8f47e0764dd6c5",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we fine with those ? Shouldn't we use Teller or something ? 👀

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hum yeah I'll remove them. I wanted to make them available for us but I'll write this in a notion page

Comment thread package.json Outdated
"scripts": {
"local": "nodemon",
"dev": "API_ENV=staging nodemon",
"dev:public": "PUBLIC_API=true API_ENV=staging nodemon",

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why not make it part of the default yarn dev ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it could be part of config. true

Comment thread server/callback.ts
} catch (e) {
console.error(e);
} catch (error) {
console.error('Error retrieving tokens ⛔️', error);

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💅 I'm not a huge fan of logs containing emojis if they end up in an actual logging system (#Datadog) 🫣

I can see the previous logs with some, but just sayin' 😁

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this project purpose is to help partner to integrate shine connect. this is an example of how to implement the api calls and the result of some endpoints

Comment thread pages/scopes.ts Outdated
@@ -0,0 +1,29 @@
export const minimalScopes = ['openid', 'profile', 'user:profile:read'];

export const availableScopes = {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't have any context about this, so I trust you.

For my knowledge though: are those bound to be kept in sync with another system ? If that's the case, are those gonna end up being pulled automatically in the future ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ysavicsh
ysavicsh force-pushed the fix-make-it-work branch 3 times, most recently from 5e2abd2 to 0116f87 Compare August 26, 2026 15:06
@ysavicsh
ysavicsh merged commit 02328fb into master Aug 26, 2026
1 check passed
@shine-bot
shine-bot Bot deleted the fix-make-it-work branch August 26, 2026 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants