use numeric slug for linking
While our link was using the numeric slug, the link text was using a pattern that resulted in a 404. This switches to use the numeric slug.
🍱 Types of changes
What types of changes does your code introduce to AEP? Put an x in the boxes
that apply
- [ ] Enhancement
- [ ] New proposal
- [ ] Migrated from google.aip.dev
- [x] Chore / Quick Fix
📋 Your checklist for this pull request
Please review the AEP Style and Guidance for contributing to this repository.
General
- [x] Basic Guidance is met.
- [x] Ensure that your PR references AEPs correctly.
- [ ] My code has been formatted
(usually
prettier -w .)
@rambleraptor , I think we moved to numeric slugs without a text-based alias? All I know is that the guidance was suggesting a text-based URL that was resulting in 404s.
Hmm, if this change is correct, than I think perhaps https://aep.dev/tooling/linter/rules/0162/ may need an update, too?
I see this guidance does still refer to word-based slugs, so possibly that deserves attention?
@toumorokoshi
I think we're trying to use the non-numeric slugs. Both of them should work regardless.
@rofrankel too
I think in this particular case we shouldn't update those - that should probably always point at not-precdent in case that aep is updated.
But those are examples, correct? so really it comes down to whether the linter accepts them (I presume it does).