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

HTML page Title tag for Guilds says "Group Plans" #12738

Closed
Alys opened this issue Oct 30, 2020 · 6 comments · Fixed by #12761
Closed

HTML page Title tag for Guilds says "Group Plans" #12738

Alys opened this issue Oct 30, 2020 · 6 comments · Fixed by #12761

Comments

@Alys
Copy link
Contributor

Alys commented Oct 30, 2020

When you are viewing a Guild that does not have a Group Plan, the title says "Group Plans".
It should say "Guilds".

The two examples below are public guilds that definitely don't have Group Plans in them.

image
image

Thanks to @citrusella for reporting this.

@Alys
Copy link
Contributor Author

Alys commented Oct 30, 2020

@thewilloftheshadow / beinginhisshadow (2eb203f5-1b1e-4e5d-84a4-22f130a1090e) has posted technical details:

"... the Group Plans does show up for me. I went looking in the code, and saw that the section is in fact set to the wrong variable for line 504 in this file:

if (this.isParty) this.searchId = 'party';
if (!this.searchId) this.searchId = this.groupId;
await this.fetchGuild();
this.$store.dispatch('common:setTitle', {
section: this.$t('groupPlans'),
subSection: this.group.name,
});
this.$root.$on('updatedGroup', this.onGroupUpdate);
},

this.$store.dispatch('common:setTitle', {section: this.$t('groupPlans'), subSection: this.group.name });

"This was changed in PR #12627 which was pulled to the server today"

@thewilloftheshadow
Copy link
Contributor

I can take this on (:

@Alys
Copy link
Contributor Author

Alys commented Oct 30, 2020

@thewilloftheshadow Fantastic, thanks. :) It's yours.

@thewilloftheshadow
Copy link
Contributor

Sorry about the delay, I'm having trouble setting up my local copy so I can test and make sure the change didn't affect anything else

@paglias
Copy link
Contributor

paglias commented Nov 3, 2020

Looks like normal guilds / parties and group plans use the same component (group.vue) so we'll have to add a check on whether you're viewing the group as a group plan or not (probably by looking at whether group-plans is in the url). Note that with #12743 merged there are a couple more places to update in the component file, make sure to work on an up to date version of the develop branch.

@thewilloftheshadow let me know if you need any help

@thewilloftheshadow
Copy link
Contributor

I think I got it all, but there is one space to check that I noted in the PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants