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.

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

Validate the footer links text #1421

Merged

Conversation

yash-learner
Copy link
Member

@yash-learner yash-learner commented Aug 28, 2023

Proposed Changes

Fixes #1399

  • Add validation in Mutators
  • Update frontend
  • Update en.yml
  • Add specs

@pupilfirst/developers

Merge Checklist

  • Add specs that demonstrate bug / test a new feature.
    - [] Check if route, query, or mutation authorization looks correct.
    - Add tests for authorization, if required.
  • Ensure that UI text is kept in I18n files.
    - [ ] Update developer and product docs, where applicable.
  • Prep screenshot or demo video for changelog entry, and attach it to issue.
    - [ ] Check if new tables or columns that have been added need to be handled in the following services:
    - Users::DeleteAccountService
    - Courses::CloneService
    - Courses::DeleteService
    - Courses::DemoContentService
    - Levels::CloneService
    - Schools::DeleteService
    - [ ] Check if changes in packaged components have been published to npm.
    - [ ] Add development seeds for new tables.
    - [ ] If the updates involve Graph mutations ensure that the files are migrated to the new approach without a mutator.

@yash-learner yash-learner marked this pull request as ready for review August 30, 2023 11:55
@yash-learner yash-learner changed the title Validate the footer links Validate the footer links text Aug 30, 2023
app/graphql/mutations/create_school_link.rb Outdated Show resolved Hide resolved
config/locales/en.yml Outdated Show resolved Hide resolved
config/locales/en.yml Outdated Show resolved Hide resolved
config/locales/en.yml Outdated Show resolved Hide resolved
spec/system/school/customization_spec.rb Outdated Show resolved Hide resolved
spec/system/school/customization_spec.rb Outdated Show resolved Hide resolved
app/frontend/shared/utils/StringUtils.res Show resolved Hide resolved
bodhish
bodhish previously approved these changes Sep 7, 2023
app/graphql/concerns/validate_school_link_title.rb Outdated Show resolved Hide resolved
Co-authored-by: Bodhish Thomas <bodhish@gmail.com>
@bodhish bodhish merged commit 0040212 into pupilfirst:master Sep 7, 2023
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.

Footer link text isn't validated or normalized
3 participants