Skip to content

Conversation

maze-runnar
Copy link
Contributor

@maze-runnar maze-runnar commented Nov 1, 2019

Fixes #3584

Short description of what this resolves:
It changes the google plus entry to google groups in admin/content page .Also fixes the required regex expression for google goups url (https://groups.google.com/forum/#!forum/open-event).Also updated the warning messages shows in box after filling wrong url from "enter a valid google plus url" to "enter a valid google groups url"

Changes proposed in this pull request:
-changed google plus entry in (https://eventyay.com/admin/content) to google groups entry.

Screenshot from 2019-10-31 18-00-54
on entering the wrong url -
Screenshot from 2019-10-31 18-19-13
on entering the right url -
Screenshot from 2019-10-31 18-19-53

Checklist

  • I have read the Contribution & Best practices Guide.
  • My branch is up-to-date with the Upstream development branch.
  • The acceptance, integration, unit tests and linter pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

@kushthedude
Copy link
Member

kushthedude commented Nov 2, 2019

@maze-runnar It looks good, I have pushed a test commit to Successfully Deploy netlify.

@kushthedude
Copy link
Member

@maze-runnar Can you also rename the Google Plus to Google Groups in the complete models and input id?
You will also need to change the icon for the same. In the meantime you can convert this PR to WIP

@maze-runnar
Copy link
Contributor Author

@maze-runnar It looks good, I have pushed a test commit to Successfully Deploy netlify.

Thanks @kushthedude

@maze-runnar
Copy link
Contributor Author

@maze-runnar Can you also rename the Google Plus to Google Groups in the complete models and input id?
You will also need to change the icon for the same. In the meantime you can convert this PR to WIP

I have changed input_id , But i am not sure about models , can you please guide me .

@kushthedude kushthedude changed the title fix : Renaming google plus entry to google groups in admin content social link page fix: Renaming google plus entry to google groups in admin content social link page Nov 2, 2019
@auto-label auto-label bot added the fix label Nov 2, 2019
@kushthedude
Copy link
Member

@maze-runnar Can you also rename the Google Plus to Google Groups in the complete models and input id?
You will also need to change the icon for the same. In the meantime you can convert this PR to WIP

I have changed input_id , But i am not sure about models , can you please guide me .

In app/models

<label>{{t 'Google Groups'}}</label>
{{widgets/forms/link-input
segmentedLink=socials.segmentedGoogleUrl
inputId='google_plus'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename here too

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@maze-runnar
Copy link
Contributor Author

@maze-runnar Can you also rename the Google Plus to Google Groups in the complete models and input id?
You will also need to change the icon for the same. In the meantime you can convert this PR to WIP

I have changed input_id , But i am not sure about models , can you please guide me .

In app/models

ok , thanks

@maze-runnar
Copy link
Contributor Author

@kushthedude sir i have made changes. Do i need to make any further changes ?

</div>
<div class="field">
<label>{{t 'Google +'}}</label>
<label>{{t 'Google Groups'}}</label>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why Plural?
Groups implies you can add multiple groups, but there is only one text field. Keep it Group

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok sir .

@kushthedude
Copy link
Member

@maze-runnar Please change groups group

{{widgets/forms/link-input
segmentedLink=socials.segmentedGoogleUrl
inputId='google_plus'
inputId='google_groups'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change here too else validation wont work

@maze-runnar
Copy link
Contributor Author

maze-runnar commented Nov 3, 2019

I have changed google groups to google group but i think its actual name is google groups not group.
As in url it is groups.google.com
Screenshot from 2019-11-03 19-10-24
**

@abhinavk96 abhinavk96 merged commit 11f639a into fossasia:development Jan 2, 2020
@maze-runnar maze-runnar deleted the patch-7 branch January 2, 2020 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename google plus entry to google groups from admin social link page

3 participants