Skip to content

Add logo to metadata collectors#2

Open
kofzera wants to merge 4 commits intoSUNET:mainfrom
kofzera:add_collector_metadata
Open

Add logo to metadata collectors#2
kofzera wants to merge 4 commits intoSUNET:mainfrom
kofzera:add_collector_metadata

Conversation

@kofzera
Copy link
Copy Markdown

@kofzera kofzera commented Mar 14, 2023

No description provided.

@c00kiemon5ter
Copy link
Copy Markdown
Member

It would be good to have the same on the OIDC side to keep things consistent.

@kofzera kofzera force-pushed the add_collector_metadata branch from 2ced137 to 8dd59fe Compare March 20, 2023 07:43
@kofzera kofzera changed the title Add logo and description to SAML metadata Add logo to metadata collectors Mar 20, 2023
@c00kiemon5ter
Copy link
Copy Markdown
Member

LGTM; if you have tested this we should merge.

@c00kiemon5ter
Copy link
Copy Markdown
Member

Any news on this?

@kofzera kofzera force-pushed the add_collector_metadata branch 2 times, most recently from 4efe674 to d0b1f6a Compare May 9, 2023 06:20
Comment thread src/swamid_plugins/metainfo/collectors/oidc.py Outdated
@kofzera kofzera force-pushed the add_collector_metadata branch 2 times, most recently from e849ceb to aa7eb31 Compare May 10, 2023 10:38
@kofzera kofzera force-pushed the add_collector_metadata branch from aa7eb31 to 9156529 Compare May 21, 2023 12:47
@kofzera kofzera force-pushed the add_collector_metadata branch from 9156529 to 85ef6c5 Compare May 21, 2023 12:56
@melanger
Copy link
Copy Markdown

@c00kiemon5ter We have made some improvements, tested this and it works for us. Feel free to merge this if you're happy with it.

@melanger
Copy link
Copy Markdown

@c00kiemon5ter We have made some improvements, tested this and it works for us. Feel free to merge this if you're happy with it.

Also, to make this fully functional, IdentityPython/SATOSA#438 needs to be merged.

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.

3 participants