Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat(ci): adding ci pipelines #32

Open
wants to merge 69 commits into
base: master
Choose a base branch
from
Open

feat(ci): adding ci pipelines #32

wants to merge 69 commits into from

Conversation

verissimor
Copy link

No description provided.

lukin and others added 30 commits August 17, 2021 10:30
@lukin
Update dependencies
e8fcdd9
@ifeelcream
Added IDP extension
5dad1e2
@lukin
Prettier
6079295
@lukin
Merge pull request #1
b3c3e3c
@lukin
Update dependencies
69dc820
@lukin
Update configuration files
bf38ae6
@lukin
Build resources
9d13778
@lukin
Update README.md
8db0ead
@lukin
Fix username validation
5a07825
@lukin
Update dependencies
766a38b
@lukin
Update target
3b44fd2
@lukin
Build resources
4e3f17b
@lukin
Rollback target
2622c84
@lukin
Update dependencies
295f5fc
@lukin
Build resources
54e9c33
@ronmrcdo
ui(totp): configure and login otp
4cff8b3 
- added configure and login otp pages
@lukin
Prettier
6067b8f
@lukin
Merge pull request lukin#9
8781594 
[pages] Add Login OTP
@lukin
[link] Remove extra styles
fef457a
@tombrouwer
Update alerts.ftl
a195d30 
Hi,

Thanks for your nice project! I learned a lot of it!

I saw that you don't apply breaks in the switch. This is necessary.

check: https://freemarker.apache.org/docs/ref_directive_switch.html
@tombrouwer
Update locales.ftl
4f79746 
Hi,

An HTML tag issue.
@lukin
[alerts] Use break in switch (lukin#11)
b86e947
@lukin
[locales] Fix HTML tag (lukin#12)
224d737
@lukin
Migrate from Snowpack to Vite
118d107
@YoanSimco
Update register.ftl
52f9892
@lukin
Merge pull request lukin#19
f319a25 
[pages] Fix Register
@lukin
[pages] Add Logout Confirm
2c92c52
@YoanSimco
Update logout-confirm.ftl
5bef95c
@YoanSimco
Apply suggestions to 1 file
ad9e375
@lukin
Merge pull request lukin#21
f5490bd 
[pages] Fix Logout
@verissimor
Copy link
Author

This PR follows the request #6 (comment):

CI/CD pipeline would be great. Something that can package up the theme into an archive, then add it to github releases. Then it can easily be installed using the Keycloak Operator.

After build, the theme is zipped in a similar approach as https://github.com/Alfresco/alfresco-keycloak-theme/releases/tag/0.3.2

Giving permission to the pipeline:
Navigate Git > Project > Settings > Actions > General and enable Workflow permissions = Read and write permissions.

In order to create a new release:

  1. Navigate to https://github.com/lukin/keywind/releases
  2. Select 'Release'
  3. Create a new/draft release with:
    • Choose a tag: 0.0.1
    • Target: master
    • Release Title: v0.0.1

Important that tag and title matches.

I hope it helps, please let me know if the commit looks good for you.

@lukin
Copy link
Owner

lukin commented Jan 15, 2023

Hey 👋

Thanks for contributing. I need some time to finish #23. After that, I will test this PR.

@verissimor
Copy link
Author

Any news here @lukin ?

@IDerr IDerr mentioned this pull request Feb 20, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.