Skip to content

Comments

tweak(web/standalone/deployer.ejs) update keymaster link to new portal#1022

Merged
tabarra merged 4 commits intocitizenfx:developfrom
Gravxd:master
Feb 23, 2025
Merged

tweak(web/standalone/deployer.ejs) update keymaster link to new portal#1022
tabarra merged 4 commits intocitizenfx:developfrom
Gravxd:master

Conversation

@Gravxd
Copy link
Contributor

@Gravxd Gravxd commented Jan 5, 2025

No description provided.

@Gravxd Gravxd requested a review from tabarra as a code owner January 5, 2025 05:14
@Misha0717
Copy link
Contributor

Dupe of #1016

@tabarra tabarra changed the base branch from master to develop January 10, 2025 07:34
@tabarra
Copy link
Collaborator

tabarra commented Jan 10, 2025

Yes, dupe, but the other is "stale" (well, one week).
If you implement the change requested I merge this one instead.

I think it's a good opportunity to include a link to the new docs page:
https://support.cfx.re/hc/en-us/articles/16539369935900-How-to-create-a-registration-key

On a quick note, small stuff like this I usually prefer just doing myself, and actually had already done the change but didn't commit.
But I'm happy to merge community PRs, and community members are usually happy to help so I'm fine with accepting PRs instead.

@Gravxd
Copy link
Contributor Author

Gravxd commented Jan 10, 2025

Didn't see another PR, mb on the dupe - link is updated as requested though

@tabarra
Copy link
Collaborator

tabarra commented Jan 14, 2025

The link was correct 😝
What I meant was also adding a link to the docs, like a For more information, read the [documentation](https://...), or even just (docs) with a link.

@Gravxd
Copy link
Contributor Author

Gravxd commented Jan 14, 2025

Medal_zf8djO3R4U
Something like this?

@Kuuzoo
Copy link

Kuuzoo commented Feb 14, 2025

Medal_zf8djO3R4U Something like this?

Love it! Think with that @tabarra can merge this one and close the old one which is stale.

@tabarra tabarra merged commit 1cc5394 into citizenfx:develop Feb 23, 2025
1 check passed
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.

4 participants